Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added sdrf from xroulab samplesheets #676

Conversation

NicolasProvencher
Copy link

Added 141 sdrf from samplessheet of manually curated pride studies

Unsure if the correct use of TMT label since our curation was made per file instead of per sample thus making all tmt asscosiated to tmt6plex through comment modification column

@sonatype-lift
Copy link

sonatype-lift bot commented Aug 4, 2023

Sonatype Lift is retiring

Sonatype Lift will be retiring on Sep 12, 2023, with its analysis stopping on Aug 12, 2023. We understand that this news may come as a disappointment, and Sonatype is committed to helping you transition off it seamlessly. If you’d like to retain your data, please export your issues from the web console.
We are extremely grateful and thank you for your support over the years.

📖 Read about the impacts and timeline

@ypriverol
Copy link
Member

@NicolasProvencher it will take is sometime to review the datasets. Impressive contribution, if you find in the meantime some info about the TMT would be amazing.

@NicolasProvencher
Copy link
Author

seems like your validation tools pulled up some problems that the one i used didnt i will get right on fixing those

@ypriverol
Copy link
Member

@NicolasProvencher some of the experiments are linking to mgf files and not have any information about the label, not the correct label. I suggest that in the name of those files you put like {PX Accession}-identification.sdtf.tsv to make clear to users of these SDRF that they can be used mainly for peptide/protein identification, but they have missing data needed for quant. What do you think?

@ypriverol
Copy link
Member

@NicolasProvencher any update on this?

@NicolasProvencher
Copy link
Author

This isnt a priority for us right now, ill cancel the merge for now and redo it once we have time to manually go back through all the study to add the missing info

@ypriverol
Copy link
Member

Thanks, I will close the PR for now.

@ypriverol ypriverol closed this Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants