populationgenomics / automated-interpretation-pipeline

Rare Disease variant prioritisation MVP

MIT License

5 stars 4 forks source link

show_default fix #383

Closed MattWellie closed 2 weeks ago

MattWellie commented 2 weeks ago

Fixes

show_default is not a valid argument to argparse - CoPilot led me astray. Looks like show_default is a valid argument for the Lua version of the library, but not for Python, so this was causing startup failures.
A while back I removed some checkpoints from the Hail process - the recent runtimes are WAY higher than I would expect, so reinstating some checkpoints to see if that alleviates the issue.

Proposed Changes

Remove bad argument from argparse
adds checkpoints back in to match a previous state (here)

Checklist

[ ] Related Issue created
[ ] Tests covering new change
[x] Linting checks pass

cassimons commented 2 weeks ago

Co-pilot did you dirty!