-
-
Notifications
You must be signed in to change notification settings - Fork 479
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue 191 python3.5 #364
Issue 191 python3.5 #364
Conversation
…o start getting notebooks tested
Codecov Report
@@ Coverage Diff @@
## master #364 +/- ##
=========================================
+ Coverage 97.32% 97.62% +0.3%
=========================================
Files 60 60
Lines 3921 4549 +628
=========================================
+ Hits 3816 4441 +625
- Misses 105 108 +3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great as is, thanks @martinjrobins . Was there anything else that needed to be done from #191? (rewriting docs?)
I don't think so, the README already states that we support Python 3.5+, so nothing to change there |
Description
Fixes #191
Key issue was that for python 3.5 (and lower) the
dict
class is unordered, so python is free to reorder things as it likes. We were assuming in a few places that the order of ourdicts
was fixed, so this was throwing errors.other issues were to do with scikits.odes:
Type of change
Please delete options that are not relevant.
Key checklist:
$ flake8
$ python run-tests.py --unit
$ cd docs
and then$ make clean; make html
You can run all three at once, using
$ python run-tests.py --quick
.Further checks: