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

Rozwalak2024 #1154

Merged
merged 12 commits into from
Apr 5, 2024
Merged

Rozwalak2024 #1154

merged 12 commits into from
Apr 5, 2024

Conversation

rozwalak
Copy link
Contributor

Pull Request

This PR is for a

For the following list(s):

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

Publication Information

This is to close

PR Workflow

  1. Open this PR with sample metadata on the samples metadata sheet (:tada: you're already here!)
  2. Wait for checks for sample metdata to pass
  3. (If checks fail) make corrections, and push changes to this branch (no need to open a new PR!)
  4. (Once passed) comment on this PR @spaam-bot please autofill <table_name> <project_id> to get a half-filled template! (may take a minute or so to get the comment with the file)
  5. Fill in the template, and verify autofilled data correct!
  6. Once filled in, append(!) the new rows from the TEMPLATE file to the end of the corresponding library metadata file, and update this PR
  7. Wait for checks for library metadata to pass
  8. Request review!

Pre-review checklist (new publications)

  • 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 and not amplicon data
    • Note: 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)?
  • Library metadata has been added
  • Changelog is updated to include the publication under 'Added'?
  • Pull request has passed validation checks (see automated comment from github-bot)?
  • (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:

@rozwalak
Copy link
Contributor Author

@jfy133 Can you help me with these problems?

@jfy133
Copy link
Member

jfy133 commented Mar 28, 2024

@rozwalak if you look closely at the first error, you'll see you have a space at teh end of your project_id string ;)

The second, you need to make a second PR to update the corresponding enum file: https://github.com/SPAAM-community/AncientMetagenomeDir/tree/master/assets/enums, as described in the error :)

Once the second PR is merged in, you can fix the samplesheet(s), and then restart the test

Copy link
Member

@jfy133 jfy133 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One main comment, otehrwise almost there, main issue is with the library table referring to raw reads not your new genbank assembly :)

@jfy133
Copy link
Member

jfy133 commented Apr 5, 2024

Thanks @rozwalak !

Copy link

github-actions bot commented Apr 5, 2024

AMDirT, version 1.5.0

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

@jfy133 jfy133 merged commit f59bd3b into master Apr 5, 2024
1 check passed
@jfy133 jfy133 deleted the Rozwalak2024 branch April 5, 2024 11:02
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