ACED-IDP / aced_etl_pod

etl worker pod
MIT License
1 stars 1 forks source link

adds submission import and removes command line calls #8

Closed matthewpeterkort closed 9 months ago

matthewpeterkort commented 9 months ago

Don't think the try, except block over everything is a good way to catch and handle errors. Tested and working with: quay.io/ohsu-comp-bio/aced-etl:latest-test Is noticeably faster than the sub process command line calls.

bwalsh commented 9 months ago

Also, we should delete the load_all script

matthewpeterkort commented 9 months ago

fixed in https://github.com/ACED-IDP/aced_etl_pod/pull/8/commits/0569f682042c4268234f5d926318c7f0800bc91f tested with- quay.io/ohsu-comp-bio/aced-etl:latest-test