Skip to content

Add: Upgrade PlotMatplot - #712

Closed
DEUCE1957 wants to merge 5 commits into
Grid2op:dev_1.12.6from
DEUCE1957:matplotlib
Closed

Add: Upgrade PlotMatplot#712
DEUCE1957 wants to merge 5 commits into
Grid2op:dev_1.12.6from
DEUCE1957:matplotlib

Conversation

@DEUCE1957

Copy link
Copy Markdown
Contributor

Extended PlotMatplot to support changing:

  • The size of the markers representing loads, generators, energy storage and busbars
  • The color map used to color markers and/or powerlines
  • Coloring support for loads and generators (interface does not exactly match BasePlot, however)
  • The normalization of the color map (can pass a matplotlib cmap name like "viridis", or a list of numbers [0.0, 1.0] to normalize the colors between)
  • The attribute used to color markers and/or powerlines (e.g. "rho" for ratio of thermal limit)
  • The attribute used to display information next to the markers / powerlines (e.g. "v" for voltage and "p" for active power)
    This included a refactor of the class using Dataclasses, which should make future changes easier to propagate.

There may be additional potential to refactor, since there is some repetition across the draw_load, draw_gen, etc.

  • Documentation
  • Jupyter Notebook
  • Additional Unit Tests

DEUCE1957 added 5 commits June 22, 2025 18:32
Signed-off-by: DEUCE1957 <2246306W@student.gla.ac.uk>
Signed-off-by: DEUCE1957 <2246306W@student.gla.ac.uk>
Signed-off-by: DEUCE1957 <2246306W@student.gla.ac.uk>
Signed-off-by: DEUCE1957 <2246306W@student.gla.ac.uk>
Signed-off-by: DEUCE1957 <2246306W@student.gla.ac.uk>
@sonarqubecloud

Copy link
Copy Markdown

@BDonnot
BDonnot changed the base branch from dev_1.11.1 to dev_1.12.1 July 29, 2025 11:41
@BDonnot
BDonnot changed the base branch from dev_1.12.1 to dev_1.12.2 August 28, 2025 18:47
@BDonnot
BDonnot changed the base branch from dev_1.12.2 to dev_1.12.6 September 8, 2026 07:34

@BDonnot BDonnot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me if it works (i'll check this and resolve conflicts)

@BDonnot BDonnot mentioned this pull request Sep 8, 2026
@BDonnot

BDonnot commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

I'm closing this one, I reopened one in #772 (this code + full CI + resolve merge conflitcs)

@BDonnot BDonnot closed this Sep 8, 2026
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.

2 participants