Skip to content

Change BatchSummaryCollector default y-axis scaling to independent axes#392

Merged
jepegit merged 1 commit into
masterfrom
cursor/batch-summary-match-axes-default-0648
Jul 1, 2026
Merged

Change BatchSummaryCollector default y-axis scaling to independent axes#392
jepegit merged 1 commit into
masterfrom
cursor/batch-summary-match-axes-default-0648

Conversation

@jepegit

@jepegit jepegit commented Jun 30, 2026

Copy link
Copy Markdown
Owner

Summary

Changes the default y-axis scaling behavior for BatchSummaryCollector from matching axes across subplots to independent axes.

Problem

The BatchSummaryCollector currently defaults to match_axes=True, which forces all subplots to share the same y-axis scale. This behavior is only suitable for specific plot combinations and is not appropriate for the majority of use cases where different variables (e.g., capacity, voltage, efficiency) have different scales.

Solution

Added _default_plotter_arguments to the BatchSummaryCollector class with match_axes set to False:

_default_plotter_arguments = {
    "match_axes": False,
}

Impact

  • Better default behavior: Subplots will now have independent y-axes by default, making it easier to view multiple variables with different scales
  • Backward compatible: Users who need matched axes can still get the old behavior by passing plotter_arguments={"match_axes": True}
  • Scoped change: Only affects BatchSummaryCollector, not other collectors like BatchCyclesCollector or BatchICACollector

Testing

The change follows the existing pattern used by other collectors in the codebase (documented in the class-level comments about three levels of argument precedence).

Closes #391

Open in Web Open in Cursor 

- Add _default_plotter_arguments to BatchSummaryCollector class
- Set match_axes to False by default for better usability
- Matching axes only suitable for specific plot combinations
- Users can still override with plotter_arguments={'match_axes': True}

Fixes #391

Co-authored-by: Jan Petter Maehlen <jepe@ife.no>
@jepegit jepegit marked this pull request as ready for review June 30, 2026 12:49
@jepegit jepegit merged commit 7987c30 into master Jul 1, 2026
10 of 14 checks passed
@jepegit jepegit deleted the cursor/batch-summary-match-axes-default-0648 branch July 1, 2026 08:39
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.

Labels set in database labels column are overwritten with autogenerated values by default BatchSummaryCollector default y-axis scaling

2 participants