MuMoT

MuMoT copied to clipboard

Would be very useful for debugging, particularly if the logging level could be set using an environment var. At present, debugging via `print` is made difficult by aggressive use of...

willfurnass

Switch to using matplotlib's object-oriented API

1

Using the stateful API (functions and attributes of the `matplotlib.pyplot` module) correctly gets increasingly tricky as a libraries that depends on matplotlib grows larger. MuMoT's use of matplotlib would be...

willfurnass

Consider pointing binder icon to latest release branch

2

Currently the binder icon in `README.md` and `getting_started.rst` point to the master branch. It may be better to have these point to the most recent release branch instead. If this...

jarmarshall

At present travis-ci.org jobs are [taking ~50 mins](https://travis-ci.org/DiODeProject/MuMoT/builds) and will likely be longer if/when PR #363 is merged: - Do all notebooks implicitly listed in Issue #325 _need_ to be...

willfurnass

Update contributors

2

This issue should be re-opened for every new release. Update contributors in: - [ ] Python docstring - [ ] `about()` - [ ] readthedocs

jarmarshall

Consider DOI release assignment strategy

1

Currently the deployment strategy (https://mumot.readthedocs.io/en/latest/development.html) specifies requesting a new DOI for each release. This may not be appropriate for patch releases, unless contributors have changed. Note, however that the ORDA...

jarmarshall

Muticontroller plot warning

1

Noted during work on pull request #350 : ``` /Users/james/Google Drive/Home/Git/MuMoT/mumot/views.py:550: MatplotlibDeprecationWarning: Adding an axes using the same arguments as a previous axes currently reuses the earlier instance. In a...

jarmarshall

Clean exceptions

3

Exceptions, such as warnings and errors, are sometimes raised with a `print("Warning message")`. They should instead be raised with `raise exceptions.MuMoTWarning("Warning message")`

joefresna

Greek letters should be used consistently in all functions

2

in single controller, greek letters work with the backslash, instead in Multicontrollers backslash should be omitted. Very hard for users to guess which is the correct format if changes from...

joefresna

For example, the following code from the user manual does not result in a multicontroller with a runs slider or aggregate results checkbox: ``` silentIntegrate2 = model4.integrate(silent = True) silentSSA2...

jarmarshall