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
decouple genotype api #772
Conversation
Removes two mip run tests because of conda environment problems
…uple-genotype-api
Hi @Mropat , this one is ready for review! |
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 have some comments
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
"code approved by" and ""Merge and deploy" approved by" is missing |
I see this all the time but we should get better on reminding each other. When you did the review @patrikgrenfeldt you did not approve or requested changes. Was there any particular reason for that? |
This PR decouples the genotype package from CG. From now on genotype will live in another conda environment and will be called on from the CLI.
How to prepare for test
bash /home/proj/production/servers/resources/hasta.scilifelab.se/update-cg-stage.sh [THIS-BRANCH-NAME]
How to test
Expected test outcome
check that the command worked
check that the samples where added
Take a screenshot and attach or copy/paste the output.
Review
Thanks for filling in who performed the code review and the test!
This version is a