Repository navigation
Conversation
magnesj
added a commit
that referenced
this pull request
Oct 25, 2026
magnesj
added a commit
that referenced
this pull request
Oct 25, 2026
…sion combos Covers CASE+WELL, VECTOR+CASE and VECTOR+WELL combinations by building plots via new_summary_plot, enabling use_common_legend and verifying curve counts. Also exports snapshots (GUI mode only) for visual verification, confirmed to correctly split into color and symbol legends for all three combinations.
magnesj
added a commit
that referenced
this pull request
Oct 25, 2026
Adds CASE+GROUP, VECTOR+GROUP and CASE+REGION scenarios, confirming the OBJECT dimension used for the split legend works for any summary object category (not just wells), as visually verified via exported snapshots.
magnesj
added a commit
that referenced
this pull request
Oct 25, 2026
…bols Pin the icon bounding rect to the full icon size, so symbols drawn smaller than the icon are not rescaled to fill the icon slot.
When the curve color is given by one summary dimension (case, vector or object such as well, group or region) and the curve symbol by another, create one color legend and one symbol legend instead of one entry per color/symbol combination.
Show the color and symbol legends as two widgets below the page title, also when exporting the page. Falls back to the merged legend otherwise.
Test common legend splitting across case, vector, well, group and region dimension combinations.
magnesj
added a commit
that referenced
this pull request
Oct 26, 2026
…bols Pin the icon bounding rect to the full icon size, so symbols drawn smaller than the icon are not rescaled to fill the icon slot.
magnesj
added a commit
that referenced
this pull request
Oct 26, 2026
When the curve color is given by one summary dimension (case, vector or object such as well, group or region) and the curve symbol by another, create one color legend and one symbol legend instead of one entry per color/symbol combination.
magnesj
force-pushed
the
simplify-summary-multiplot-legend
branch
from
October 26, 2026 06:34
50b216f to
7be2f44
Compare
magnesj
added a commit
that referenced
this pull request
Oct 26, 2026
Show the color and symbol legends as two widgets below the page title, also when exporting the page. Falls back to the merged legend otherwise.
magnesj
added a commit
that referenced
this pull request
Oct 26, 2026
Test common legend splitting across case, vector, well, group and region dimension combinations.
Member
Author
DocumentationPage: plot-window/MultiPlots.md
|
magnesj
added a commit
that referenced
this pull request
Oct 26, 2026
magnesj
force-pushed
the
simplify-summary-multiplot-legend
branch
from
October 26, 2026 13:39
4852480 to
3d35a5b
Compare
This branch has not been deployed
This file contains hidden or 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
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.
Summary
When a summary multi plot uses a common legend, the legend shows one entry per color/symbol combination, e.g. vectors x cases. If the curve color is given by one summary dimension (case, vector or object, e.g. well, group or region) and the curve symbol by another, the common legend is now split into:
WOPR, WOPT)The dimensions are found from the summary curves (
RimPlot::visibleCurvesForLegend()) usingRiaSummaryAddressAnalyzer. Qwt legend items are not inspected. Falls back to the existing merged legend for other curve types (e.g. ensembles) or when no color/symbol dimension pair is found.Also fixes oversized symbols in legend icons:
QwtGraphicderives its bounding rect from the painted content, so the bounding rect is now pinned to the full icon size.Fixes #15050