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
delly: 0.8.2 -> 0.8.5 #99128
delly: 0.8.2 -> 0.8.5 #99128
Conversation
I run through a whole genome bam file, it resulted in 30k+ lines of vcf file. I will benchmark against gold standard tomorrow, I think however, that this can be merged after the license has been updated. Thanks a lot for providing this update! |
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.
Seems the license has changed from gpl3 to bsd2, @jbedo , could you update this as well?
dellytools/delly@06f0b0c#diff-04c6e90faac2675aa89e2176d2eec7d8R12
Oops, completely missed the license change! |
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.
Hm... I think it is actually BSD3 isn't it, not mit? If you can change to bsd3 it would be nice :-)
Not sure how I got that wrong, I meant to write bsd3. |
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.
Nice! Thanks :-)
@ryantm could you have a look at this one? As far as I can tell everything is ok. I have also tested running the tool locally. |
Result of 1 package built:- delly |
Motivation for this change
New release of delly.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)