-
Notifications
You must be signed in to change notification settings - Fork 33
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
Rozwalak2024 #1154
Conversation
@jfy133 Can you help me with these problems? |
@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 Once the second PR is merged in, you can fix the samplesheet(s), and then restart the test |
There was a problem hiding this 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 :)
ancientsinglegenome-hostassociated/libraries/ancientsinglegenome-hostassociated_libraries.tsv
Show resolved
Hide resolved
…genomeDir into Rozwalak2024
ancientsinglegenome-hostassociated/samples/ancientsinglegenome-hostassociated_samples.tsv
Outdated
Show resolved
Hide resolved
…-hostassociated_samples.tsv
Thanks @rozwalak ! |
AMDirT, version 1.5.0 SamplesAncient Metagenome Host Associated
Ancient Single Genome Host Associated
Ancient Metagenome Environmental
LibrariesAncient Metagenome Host Associated
Ancient Single Genome Host Associated
Ancient Metagenome Environmental
|
Pull Request
This PR is for a
For the following list(s):
Publication Information
This is to close
PR Workflow
@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)Pre-review checklist (new publications)
github-bot
)?/assets/enums
, e.g. material or archive)? If so the corresponding PR is here: