search

holgerBerger / hpc-workspace

Automatically exported from code.google.com/p/hpc-workspace
GNU General Public License v3.0
18 stars 13 forks source link

Cara improvements #106

Closed casio888 closed 9 months ago

casio888 commented 10 months ago

Improvements from use on CARA:

This contains the 2 commits from #105 , which may be bad form?

holgerBerger commented 10 months ago

https://github.com/holgerBerger/hpc-workspace/pull/106/commits/c6f088293b7544df49cd6ace28b94196b08b970e is I think bad, as this is by intention, if the python path is set in config to be able to find the yaml import, this order of imports is needed! same applies to ws_expirer and other python based tools

https://github.com/holgerBerger/hpc-workspace/pull/106/commits/c731ddeebbbf895f65c90d2e6f289ee1b7c47998 I do not understand, sure this does not have side effects? I think we had uninitialized memory around that flag before, is that another possible location?