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
strelka: init 2.9.5 #44250
strelka: init 2.9.5 #44250
Conversation
It would be preferable to package the dependncies in |
I agree in general, however in this case the awkward patch would still remain (albeit applied at a different time) as it is hardcoded into pyflow. It would obviate the need to recompile the file after the patch, though. As for splitting out the packages in redist, some of them already do exist in nixpkgs. However, in the interest of reproducible science I'm hesitant to use this over the bundled code: when we write in a paper that strelka 2.9.5 was used to call variants, it should be as close to the release of Illumina as possible. Is it preferable to split out the compilation of the redist packages as separate expressions in a let binding for the strelka package and then hack the cmake system to use these instead? I'm not sure this improves anything, it's just making nix build them instead of cmake. |
@GrahamcOfBorg build strelka |
Failure on aarch64-linux (full log) Attempted: strelka Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: strelka Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: strelka Partial log (click to expand)
|
I have changed platforms to only x86_64-linux. |
Motivation for this change
Strelka init at version 2.9.5.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)