search

payu-org / payu

A workflow management tool for numerical models on the NCI computing systems
Apache License 2.0
18 stars 26 forks source link

Removed scipy from dependencies #251

Closed marshallward closed 4 years ago

marshallward commented 4 years ago

Scipy is listed as a dependency but does not appear to be used in payu.

Scipy depends on NumPy, which is a very large package, so we probably do not want this to be installed.

aidanheerdegen commented 4 years ago

Well spotted.

You can just merge stuff like this if you want @marshallward

marshallward commented 4 years ago

Bob and Alistair have a interesting policy where they are not permitted to merge their own code, and the other must merge it.

I don't think I'm back up to speed yet to review yours and hold up essential NCI business, but I like the idea of someone else merging my code at the moment :).