Skip to content

[Hotfix]: issue with plots when both model and grouping are present#196

Merged
Schiano-NOAA merged 3 commits intomainfrom
hotfix-issue-188
Feb 27, 2026
Merged

[Hotfix]: issue with plots when both model and grouping are present#196
Schiano-NOAA merged 3 commits intomainfrom
hotfix-issue-188

Conversation

@Schiano-NOAA
Copy link
Collaborator

Addressing the issue brough up in #188

An interaction between model and group was introduced in the fill and line aesthetics of the plot. We could not find a solution to renaming the legend title and decided to not do a manual adjustment to the columns to change the labels.

@Schiano-NOAA Schiano-NOAA linked an issue Feb 25, 2026 that may be closed by this pull request
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

New version checklist

  • Package version in DESCRIPTION has been updated
  • Release notes have been drafted/published
  • Cheatsheet content has been updated (if applicable)
  • Cheatsheet version has been updated

@github-actions
Copy link
Contributor

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
71.0% 1:0.1 4m42s

Code coverage of files in pull request scope (87.1%)

Files Coverage
R/utils_plot.R 87.1%

Reported by octocov

@Schiano-NOAA
Copy link
Collaborator Author

This will be included with the release notes when #195 is merged

@Schiano-NOAA Schiano-NOAA merged commit ad58fbf into main Feb 27, 2026
8 checks passed
Schiano-NOAA added a commit that referenced this pull request Feb 27, 2026
…196)

* add interaction to timeseries plot line

* remove duplicated plot_timeseries

* update documentation for plot timeseries
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.

🐛 in plot_timeseries() with multiple models

1 participant