-
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
Release v24.9.1 #502
Merged
Merged
Release v24.9.1 #502
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…files added for single pulse and multi-pulse HPPC data fitting, a thermal submodel is also added. Folder created to keep data files for running example cases.
…7-add-thermal-thevenin-example-remove-params-from-others
…ers' of https://github.com/pybop-team/PyBOP into 437-add-thermal-thevenin-example-remove-params-from-others
…for performance, small fixes/formating to notebooks
Main -> develop
…st transformation arg in scipy_optimisers.py, updates transformation integration tests
…early-gmt Shift scheduled test workflow
… call to parameters.join(). Updts. fail_gradient within setter.
…-to-readme Updates to README.md
…r-gaussianloglikelihood
…le-remove-params-from-others # Conflicts: # examples/notebooks/equivalent_circuit_identification.ipynb
…uild()` initial_state dict keys to docstring
…or-gaussianloglikelihood Transformation fixes
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #502 +/- ##
=======================================
Coverage 99.04% 99.04%
=======================================
Files 52 52
Lines 3546 3560 +14
=======================================
+ Hits 3512 3526 +14
Misses 34 34 ☔ View full report in Codecov by Sentry. |
…arams-from-others
I've confirmed the testing suite passes when this release branch is installed locally via pip. It's now ready for review. |
BradyPlanden
requested review from
martinjrobins,
NicolaCourtier and
agriyakhetarpal
September 16, 2024 12:26
agriyakhetarpal
approved these changes
Sep 16, 2024
updates: - [github.com/astral-sh/ruff-pre-commit: v0.6.4 → v0.6.5](astral-sh/ruff-pre-commit@v0.6.4...v0.6.5)
chore: update pre-commit hooks
…ple-remove-params-from-others Notebook example files added for ECM and folder structure updated.
…-IITKGP docs: add Dibyendu-IITKGP as a contributor for example
…nsformations Fixes for LogPosterior w/ GaussianLogLikelihood
# Conflicts: # noxfile.py
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Release v24.9.1.
Remove optional
[cite]
dependency from PyBaMM as per this issue. This will be added back in a future patch release once PyBaMM releases their v24.9.1 patch.Issue reference
Fixes # (issue-number)
Review
Before you mark your PR as ready for review, please ensure that you've considered the following:
Type of change
Key checklist:
$ pre-commit run
(or$ nox -s pre-commit
) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)$ nox -s tests
$ nox -s doctest
You can run integration tests, unit tests, and doctests together at once, using
$ nox -s quick
.Further checks:
Thank you for contributing to our project! Your efforts help us to deliver great software.