-
Notifications
You must be signed in to change notification settings - Fork 193
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
Use RefGenie for pipeline template genomes.conf
config
#1084
Comments
Note that the RefGenie asset identifiers will be new, but hopefully we can keep the top level |
Okay so we want to generate the Okay but for this to be useful we first have to add all (or at least most) of the genomes & assets we already have to refgenie. So I will probably prioritize #1086 for now and see how easy that is. |
Yes exactly, I think that this makes most sense 👍🏻 I had thought that if #1086 took a while then we could get started with this code for just a few that are there with the rest commented out temporarily. But priority is definitely #1086 whilst there is still stuff there to be done. I guess getting the CI test written is the main task here really, as the template file should be fairly trivial as all of the hard work is done in RefGenie. |
Yup agree, and hopefully even the CI test won't be too difficult. Once all the assets are on the refgenie server, it should be straightforward |
Split from #592 - specifically #592 (comment)
So the idea would be to make a copy of the template
conf/igenomes.conf
file (need to rename to justgenomes.conf
) that is calledconf/igenomes.conf.tpl
. Then build that with the asset identifiers as described above.Once that's done, we run
refgenie populate
on that file to create theconf/genomes.conf
file with the absolute assets. This file should never be directly edited.To ensure this, we should have a CI test that regenerates the file and checks that it matches what is committed to the repo (eg.
git diff
doesn't return anything).Many / most of the reference genome assets that we currently have will be missing for now I guess, so this ties in with issue #1086 to create those. But we can leave them commented out and in a branch for now until they are ready.
The text was updated successfully, but these errors were encountered: