-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add PyBaMM version matrix to scheduled tests #123
Comments
Just to clarify – as mentioned in the comment, each PyBaMM version should correspond to a Python version? Or did you mean testing all four PyBaMM versions with the latest Python version? This should be doable by checking out the PyBaMM repository from a particular tag and building it from source. The PyPI wheels corresponding to a version can be used directly too, as long as we have support for required platforms and architectures (should be all except M-series macOS). |
Hi Agriya, Good question, I hadn't fleshed out the concept much in the original post. The goal is to ensure that a list of recent pybamm versions is supported within pybop. Ideally this would allow us to relax the pybamm version constraint and/or have a metric to decide when to bump the dependency. Creating an implementation for the latest Python and building from source (or wheels, if applicable) sounds like a good compromise between test time and coverage. That said, I'm not opposed to testing against the Python matrix we currently have in the scheduled tests. |
Testing against the Python matrix, even though it would consume a lot of parallel jobs (four PyBaMM versions per Python version), sounds like the best way to go forward with. We can do both things: use the wheels for now while building from source for M-series. We may switch to using the wheels for everything when I finish pybamm-team/PyBaMM#3772. |
Coverage is already up for this, this is just for PyBaMM 23.5
Feature description
Add a testing matrix of the most recent pybamm versions to the scheduled daily workflow. The last four releases is probably a good place to start. This should only be added to the most recent python version to avoid clashes.
Motivation
Provides us with a range of supported pybamm versions and should improve ease-of-use for end-users.
Possible implementation
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: