search

Lucy-Forrest-Lab / hm_analysis_tool

A set of tools to analyze homology models obtained with MODELLER
3 stars 2 forks source link

wrapping python #1

Open leej3 opened 4 years ago

leej3 commented 4 years ago

https://github.com/vanessa-leone/hm_analysis_tool/blob/40bf397c86d076cb3c2ebdc67e66da08d0fa81cd/hm_analysis_tool/wrapper_example.sh#L1

Ideally you should use argparser or click cli or some other python package to generate a command line interface. no need to use bash here

leej3 commented 4 years ago

More specifically, take a look at this: https://realpython.com/python-main-function/

vanessa-leone commented 4 years ago

Thanks for you suggestions. I have added a main function to execute only if I am running from the command line. I have defined the variables to run the main function, so I can do testing. I hope that now is better. I have also cleaned the directory and added installation instructions in the README.md I have pushed all the changes, and hopefully for next push I'll have some tests

On Sun, Dec 8, 2019 at 4:23 PM john lee notifications@github.com wrote:

https://github.com/vanessa-leone/hm_analysis_tool/blob/40bf397c86d076cb3c2ebdc67e66da08d0fa81cd/hm_analysis_tool/wrapper_example.sh#L1

Ideally you should use argparser or click cli or some other python package to generate a command line interface. no need to use bash here

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/vanessa-leone/hm_analysis_tool/issues/1?email_source=notifications&email_token=ANFTMECAROSGPZLTZ7P3HLTQXVQTPA5CNFSM4JYATKOKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H65NCCA, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANFTMECPBTL5WNZSLYU64X3QXVQTPANCNFSM4JYATKOA .