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

Madison2023 #1123

Merged
merged 9 commits into from
Dec 1, 2023
Merged

Madison2023 #1123

merged 9 commits into from
Dec 1, 2023

Conversation

DianaSpurite
Copy link
Contributor

@DianaSpurite DianaSpurite commented Oct 9, 2023

Adding Maddison2023 samples

Pull Request

This PR is for a

For the following list(s):

  • ancientmetagenome-environmental (README)
  • ancientmetagenome-hostassociated (README)
  • ancientsinglegenome-hostassociated (README)

New Publication

Publication Information

This pull request is to add samples from the following publication(s): 10.1371/journal.pone.0291540

This is to close #1116

Checklist

  • Publication is published (preprints currently not accepted)?
  • Checked the publication is not already in the database?
  • Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
  • Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
  • Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
  • Changelog is updated to include the publication under 'Added'?
  • Pull request has passed validation checks (see automated comment from github-bot)?
  • Review requested from @SPAAM-community/ancientmetagenomedir?
  • (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here: Madison_enums_sample_host.json #1121

@DianaSpurite DianaSpurite changed the title Update ancientmetagenome-hostassociated_samples.tsv Madison2023 Oct 9, 2023
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

AMDirT, version 1.4.6

Samples

Ancient Metagenome Host Associated

Invalid dataset ancientmetagenome-hostassociated_samples.tsv

Error Source Column Row Message
Dataset Parsing Error ancientmetagenome-hostassociated_samples.tsv - - Error tokenizing data. C error: Expected 16 fields in line 1403, saw 17

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_samples.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_samples.tsv is valid

Libraries

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_libraries.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_libraries.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_libraries.tsv is valid

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

AMDirT, version 1.4.6

Samples

Ancient Metagenome Host Associated

Invalid dataset ancientmetagenome-hostassociated_samples.tsv

Error Source Column Row Message
Dataset Parsing Error ancientmetagenome-hostassociated_samples.tsv - - Error tokenizing data. C error: Expected 16 fields in line 1403, saw 17

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_samples.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_samples.tsv is valid

Libraries

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_libraries.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_libraries.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_libraries.tsv is valid

Added Madison 2023
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

AMDirT, version 1.4.6

Samples

Ancient Metagenome Host Associated

Invalid dataset ancientmetagenome-hostassociated_samples.tsv

Error Source Column Row Message
Dataset Parsing Error ancientmetagenome-hostassociated_samples.tsv - - Error tokenizing data. C error: Expected 16 fields in line 1403, saw 17

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_samples.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_samples.tsv is valid

Libraries

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_libraries.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_libraries.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_libraries.tsv is valid

@DianaSpurite
Copy link
Contributor Author

As I understood the library protocol the authors used is this: https://www.neb.com/en/-/media/nebus/files/manuals/manuale7103-e7645.pdf?rev=de09eaf8fcdf45e0ac8a66bf6fee75fb&hash=FC51B96E7568B4482CF1B5217EF04E31

I didn't find anything familiar to library polymerase there

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

AMDirT, version 1.4.6

Samples

Ancient Metagenome Host Associated

Invalid dataset ancientmetagenome-hostassociated_samples.tsv

Error Source Column Row Message
Schema Validation Error Lithobates pipiens sample_host 1403 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1404 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1405 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1406 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1407 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1408 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1409 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1410 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1411 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1412 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]
Schema Validation Error Lithobates pipiens sample_host 1413 'Lithobates pipiens' is not an accepted value.
Please check [link=https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json]https://spaam-community.github.io/AncientMetagenomeDir/assets/enums/sample_host.json[/link]

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_samples.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_samples.tsv is valid

Libraries

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_libraries.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_libraries.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_libraries.tsv is valid

@DianaSpurite
Copy link
Contributor Author

Quick comment

  • the host species are updated to a newer genus, so Lithobates pipiens instead of Rana pipiens :Madison_enums_sample_host.json #1121

  • validation checks should pass then, once that PR is merged into master

Copy link

AMDirT, version 1.4.6

Samples

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_samples.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_samples.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_samples.tsv is valid

Libraries

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_libraries.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_libraries.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_libraries.tsv is valid

Copy link

AMDirT, version 1.4.6

Samples

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_samples.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_samples.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_samples.tsv is valid

Libraries

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_libraries.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_libraries.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_libraries.tsv is valid

@alexhbnr
Copy link
Collaborator

Hi @jfy133,
I have reviewed the PR and made some changes to it. Mainly, I have added two samples that were previously missing for a reason I am not aware.
So maybe you just briefly check whether these changes look correct to you and then I merge the PR into the master branch.

@DianaSpurite
Copy link
Contributor Author

I have added two samples that were previously missing for a reason I am not aware.

I think those were very modern samples, if I'm not wrong.

@alexhbnr
Copy link
Collaborator

Thanks @DianaSpurite, I will check that.

@alexhbnr
Copy link
Collaborator

You are right @DianaSpurite , these samples were only sampled in 1993 and 2012. I guess we can discard these two samples.
Any objections @jfy133 ?

@jfy133
Copy link
Member

jfy133 commented Dec 1, 2023

No, that's how we've done it for young samples in other publications, so discarding is fine.

The only exception is the environmental samples when you have distinct/defined sequence.

Copy link

github-actions bot commented Dec 1, 2023

AMDirT, version 1.4.6

Samples

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_samples.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_samples.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_samples.tsv is valid

Libraries

Ancient Metagenome Host Associated

ancientmetagenome-hostassociated_libraries.tsv is valid

Ancient Single Genome Host Associated

ancientsinglegenome-hostassociated_libraries.tsv is valid

Ancient Metagenome Environmental

ancientmetagenome-environmental_libraries.tsv is valid

@alexhbnr alexhbnr merged commit 09a0f38 into master Dec 1, 2023
1 check passed
@alexhbnr alexhbnr deleted the Madison2023 branch December 1, 2023 11:22
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.

Madison 2023
3 participants