Update Raxml - Githubissues

Kincekara commented 3 months ago

Closes #953 I separated Raxml and raxml-ng into different images as they should be. This PR should resolve all the issues mentioned in #953

Pull Request (PR) checklist:

[x] Include a description of what is in this pull request in this message.
[x] The dockerfile successfully builds to a test target for the user creating the PR. (i.e. docker build --tag samtools:1.15test --target test docker-builds/samtools/1.15 )
[x] Directory structure as name of the tool in lower case with special characters removed with a subdirectory of the version number (i.e. spades/3.12.0/Dockerfile)
- [ ] (optional) All test files are located in same directory as the Dockerfile (i.e. shigatyper/2.0.1/test.sh)
[x] Create a simple container-specific README.md in the same directory as the Dockerfile (i.e. spades/3.12.0/README.md)
- [ ] If this README is longer than 30 lines, there is an explanation as to why more detail was needed
[x] Dockerfile includes the recommended LABELS
[x] Main README.md has been updated to include the tool and/or version of the dockerfile(s) in this PR
[x] Program_Licenses.md contains the tool(s) used in this PR and has been updated for any missing

erinyoung commented 2 months ago

The tests look good:


#9 1.647 (Chicken:1.794547,(((Cow:0.303295,(Seal:0.362885,Mouse:0.683504):0.189557):0.139758,Whale:0.243361):0.354493,Human:1.578070):0.698172,(Frog:0.558567,Loach:1.105978):0.187173);


#12 1.422 (Lemur:0.32537910426836053057,(Saimiri:0.43157643956775904037,((M_sylvanus:0.07088921537201800516,(M_fascicul:0.05523388801310591401,(Macaca:0.01608115262138561444,M_mulatta:0.02239621504004839225):0.03388122119428586820):0.04268050828482932574):0.23505406685566176961,(Gibbon:0.16511638061802774713,((Gorilla:0.05879059075376137644,(Chimp:0.05900565905273858630,H_sapiens:0.04840728794905584109):0.02375800215551397351):0.07400567918350323127,Orangutan:0.13273893510289544451):0.04406182262600038396):0.10886147176113297741):0.10360382224976649734):0.24821952513486267300,Tarsius:0.47478261882585243159):0.0;

erinyoung commented 2 months ago

This looks great! It also looks like you put a lot of work into it.

Kincekara commented 2 months ago

@erinyoung I made the changes. I think this is ready.

erinyoung commented 2 months ago

Thank you for putting these together!

The two deployments can be found at https://github.com/StaPH-B/docker-builds/actions/runs/9502319996 https://github.com/StaPH-B/docker-builds/actions/runs/9502326488

StaPH-B / docker-builds

Update Raxml #986