Closed kotfic closed 8 years ago
lgtm :+1: (Didn't try to install thus +1 only)
@kotfic ran install as well. I can import minerva, not sure if you want me to verify anything else.
@aashish24 nope, just wanted to make sure you were happy with the metadata, if this does get installed on pypi it will be install-able as minerva.geo. minerva is taken by another project (https://pypi.python.org/pypi/Minerva/11.7.23)
metadata looks good (anyone else has any suggestions?). I like minerva.geo too.
LGTM
This adds a skeleton for minerva as an install-able package. There are several different components that properly belong outside of the girder environment, especially in the context of distributed job execution - where workers may need code which cannot otherwise be located by python.
Currently this does nothing, but i wanted to get @aashish24 @jbeezley and @mgrauer's review as a separate PR from the analysis PR. If this looks good i will merge and then pull changes into https://github.com/Kitware/minerva/pull/345 before moving forward with remote analysis execution.
ref: https://github.com/Kitware/minerva/issues/347