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
rtg-tools: init at 3.12.1 #230394
rtg-tools: init at 3.12.1 #230394
Conversation
New package for genetics : helps compared variants called by a pipeline to a reference. The only tests not passing are due to a difference in float representation (same value but with different number of digits after decimal). For best results (quality and running time), the authors advice to use RTGtools vcfeval as the engine. So this package requires NixOS#230394
For best results (quality and running time), the authors advice to use RTGtools vcfeval as the engine. Depends on NixOS#230394
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3701 |
For best results (quality and running time), the authors advice to use RTGtools vcfeval as the engine. Depends on NixOS#230394
Thanks a lot for the review. I've formatted the packages using nixfmt-rfc-style, removed the patch (now useless) and use ~/.config/rtg as a configuration directory. The package is now in |
For best results (quality and running time), the authors advice to use RTGtools vcfeval as the engine. Depends on NixOS#230394
For best results (quality and running time), the authors advice to use RTGtools vcfeval as the engine. Depends on NixOS#230394
# Use a location outside nix (must be writable) | ||
sed -i '/USER_JAVA_OPTS=$RTG_JAVA_OPTS/a mkdir -p $HOME/.config/rtg-tools' $out/bin/rtg | ||
sed -i 's|$THIS_DIR/rtg.cfg|$HOME/.config/rtg-tools/rtg.cfg|g' $out/bin/rtg | ||
# Use nix java | ||
sed -i 's|RTG_JAVA="java".*|RTG_JAVA="${jdk}/lib/openjdk/bin/java"|' $out/bin/rtg |
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.
Since we just do simple string replacement here, we should use substituteInPlace with --replace-fail to notice, if the pattern no longer matches.
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.
I've switched to substituteInPlace, except for appending to a string that I only know how to do it with sed :/
A subset of RTG Core that includes several useful utilities for dealing with VCF files and sequence data. Mostly used for vcfeval to compare VCF.
Thanks for the feedback. Commits have been squashed, waiting for approval :) |
For best results (quality and running time), the authors advice to use RTGtools vcfeval as the engine. Depends on NixOS#230394
A subset of RTG Core that includes several useful utilities for dealing with VCF files and sequence data. Mostly used for vcfeval to compare VCF.
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)