-
Notifications
You must be signed in to change notification settings - Fork 90
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
Use custom main instead of PhysicsModel in test-communications #1983
Conversation
Avoids error from Pvode solver which checks whether anything is being evolved. Also generally neater. Also adds test-communications to the CMake build.
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.
Thanks, that fixes it 👍
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.
Thanks 👍
Failing on Travis because the python f-strings require 3.6, and Ubuntu 16 is stuck on 3.5. We could upgrade Travis to use Bionic (Ubuntu 18), which has python 3.6, but then we'd be dropping support for everyone else too. I'm not opposed to doing that, but we probably want some wider discussion. Please could you change this to use |
f-strings are only supported by Python>=3.6
Thanks @johnomotani ! 👍 |
Tests have passed, just not reported correctly |
Avoids error from Pvode solver which checks whether anything is being evolved - partial fix for #1982. Also generally neater not to use
PhysicsModel
when we don't actually need time-evolution.Also adds
test-communications
to the CMake build.Most of the lines changed are just whitespace due to a change of indentation in updating
test-communications.cxx
.