test-datasets icon indicating copy to clipboard operation
test-datasets copied to clipboard

Openms idxml

Open rolivella opened this issue 2 years ago • 7 comments

This is for uploading data to test OpenMS idXML module

rolivella avatar Oct 11 '22 09:10 rolivella

Hi @rolivella , thanks for adding the test data! Could you have a look at the README.md and add the data in the corresponding subfolder (e.g. proteomics)? Also a short description of where the data comes from

ggabernet avatar Oct 11 '22 10:10 ggabernet

@ggabernet I just added the documentation and pull request

rolivella avatar Oct 11 '22 12:10 rolivella

@JoseEspinosa I already made your requested changes. Could you merge please?

rolivella avatar Oct 11 '22 13:10 rolivella

Hi @rolivella, checking this PR again, we try to keep an orderly folder structure, so that it will be easy to find the test data by everyone. It seems like this is the first proteomics dataset! 🎉 I would be therefore great to add it to a new directory called: "proteomics", and a subdirectory with the name of the species that the data comes from. See the other directory structure for e.g. genomics for an inspiration.

ggabernet avatar Oct 20 '22 07:10 ggabernet

Hi @ggabernet I think that there's already a folder called "proteomics" here:

https://github.com/nf-core/test-datasets/tree/modules/data/proteomics

But is not following the structure you mentioned with the species and so on. What should I do?

rolivella avatar Oct 20 '22 09:10 rolivella

In that case you can add a subdirectory under proteomics with the species name, and add your test data underneath. So that other proteomics datasets can follow the recommended structure 😄

ggabernet avatar Oct 20 '22 11:10 ggabernet

@ggabernet ok! I'll do it and let you know!

rolivella avatar Oct 20 '22 13:10 rolivella

Great, looks good to me then! @JoseEspinosa is there any other changes that you wanted there?

ggabernet avatar Oct 24 '22 11:10 ggabernet