search

coecms / ncimonitor

Extended usage information for HPC
1 stars 0 forks source link

Add conda packaging #10

Closed ScottWales closed 6 years ago

ScottWales commented 6 years ago

Add packaging so ncimonitor can be added to conda environments

aidanheerdegen commented 6 years ago

Thanks for that @ScottWales. It was on my to-do list for conda installability.

aidanheerdegen commented 6 years ago

Moving the command line argument parsing etc into main() has broken the (arguably incorrect) global scope of the project variable.

Traceback (most recent call last):
  File "/home/502/aph502/.local/bin/ncimonitor", line 11, in <module>
    load_entry_point('ncimonitor===dev', 'console_scripts', 'ncimonitor')()
  File "/home/502/aph502/.local/lib/python2.7/site-packages/ncimonitor/nci_monitor.py", line 334, in main
    plot_usage(db,year,quarter,plot_by_user,total_grant,users,args.pdf)
  File "/home/502/aph502/.local/lib/python2.7/site-packages/ncimonitor/nci_monitor.py", line 160, in plot_usage
    title = "Usage for Project {} on {} ({}.{})".format(project,system,year,quarter)
NameError: global name 'project' is not defined
ScottWales commented 6 years ago 
ncimonitor -P w35 --showtotal

is now working for me

aidanheerdegen commented 6 years ago

Confirmed for me too.