search

choderalab / pinot

Probabilistic Inference for NOvel Therapeutics
MIT License
15 stars 2 forks source link

added back cli #49

Closed yuanqing-wang closed 4 years ago

yuanqing-wang commented 4 years ago

this would enable easier scaling up

yuanqing-wang commented 4 years ago

@dnguyen1196

hey Duc you wanna maybe provide one for your generative as well? or make this better.

this would allow us to write simple bash loops and run on cluster, making parallel testing of models a lot easier

lgtm-com[bot] commented 4 years ago

This pull request introduces 3 alerts and fixes 2 when merging 42cd2116c5c5fdcb61bf9a77feecfb7bb17137d7 into c645cb377e0676b21b58453d66b0a63de5364a62 - view on LGTM.com

new alerts:

fixed alerts:

dnguyen1196 commented 4 years ago

@dnguyen1196

hey Duc you wanna maybe provide one for your generative as well? or make this better.

this would allow us to write simple bash loops and run on cluster, making parallel testing of models a lot easier

You're referring to pinot/app/cli.py? Looks interesting, I'll definitely take a look and implement something similar for generative model. I already have a script that one can invoke with cli arguments in https://github.com/choderalab/pinot/blob/master/scripts/generative/gvae_exp.py so hopefully that wont take much time