search

JGCRI / cassandra

Human-earth system multi-scale model coupling framework
Other
6 stars 3 forks source link

Include tethys component #32

Closed calebbraun closed 5 years ago

calebbraun commented 5 years ago

Add a water withdrawal downscaling component that calls tethys.

Resolve #15

rplzzz commented 5 years ago

It looks like this one is failing its tests because it can't locate the gcam_reader package.

Update: It looks like you may have to add the --process-dependency-links to enable this feature.

More info: https://pip.pypa.io/en/stable/reference/pip_install/#cmdoption-process-dependency-links https://stackoverflow.com/a/13587734

rplzzz commented 5 years ago

Gah! Failing because Xanthos doesn't support Python 3.6. Is that being updated on the Xanthos side?

rplzzz commented 5 years ago

@calebbraun & @crvernon It also looks as though pip is still reporting --process-dependency-links as deprecated. I wonder if we should just publish them in PyPi. We don't normally publish to CRAN because its requirements can be a little onerous (particularly the max package size), but it looks like PyPi is a lot looser in that respect. What do you guys think?

calebbraun commented 5 years ago

I like the idea of publishing our models to PyPi. I can see two problems that we would need to think about though. First, they don't support organizational accounts, so we would have to be very careful about keeping package ownership distributed among multiple people. Second, there seems to be a semi-flexible 60MB upload limit, meaning we probably couldn't ship example data.