Skip to content

Add 'get_summary_variables' function to SummaryVariables - #4824

Merged
rtimms merged 3 commits into
pybamm-team:developfrom
pipliggins:4765-sum-var-plotting
Feb 10, 2025
Merged

Add 'get_summary_variables' function to SummaryVariables#4824
rtimms merged 3 commits into
pybamm-team:developfrom
pipliggins:4765-sum-var-plotting

Conversation

@pipliggins

@pipliggins pipliggins commented Feb 7, 2025

Copy link
Copy Markdown
Contributor

Adds a function to return a dictionary of all the summary variables + cycle number from the SummaryVariables class

Fixes #4765

@codecov

codecov Bot commented Feb 7, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.70%. Comparing base (b8877b4) to head (bb22d8a).
Report is 91 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #4824   +/-   ##
========================================
  Coverage    98.70%   98.70%           
========================================
  Files          303      303           
  Lines        23312    23314    +2     
========================================
+ Hits         23009    23011    +2     
  Misses         303      303           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pipliggins
pipliggins marked this pull request as ready for review February 7, 2025 16:15
@rtimms
rtimms enabled auto-merge (squash) February 10, 2025 09:58
@rtimms
rtimms merged commit 2e9caac into pybamm-team:develop Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a method to get a dict of _all_ summary variables

2 participants