Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Configurable plotting resolution #803

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

jfrost-mo
Copy link
Member

@jfrost-mo jfrost-mo commented Aug 20, 2024

Contribution checklist

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

  • Documentation has been updated to reflect change.
  • New code has tests, and affected old tests have been updated.
  • All tests and CI checks pass.
  • Ensured the pull request title is descriptive.
  • Conda lock files have been updated if dependencies have changed.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

@jfrost-mo jfrost-mo force-pushed the configurable_plotting_resolution branch from de3f283 to 188113c Compare August 20, 2024 09:59
Copy link
Contributor

github-actions bot commented Aug 20, 2024

Coverage

@jfrost-mo jfrost-mo force-pushed the configurable_plotting_resolution branch 3 times, most recently from 5b59f5e to 7dfd564 Compare August 20, 2024 10:36
Set plotting resolution from workflow config via rose edit. The plot
resolution is then read via a function. This keeps importing the module
from having side effects.

Setting and getting plot resolution is tested.
@jfrost-mo jfrost-mo force-pushed the configurable_plotting_resolution branch from 7dfd564 to cfc9303 Compare August 20, 2024 10:37
@jfrost-mo jfrost-mo marked this pull request as ready for review August 20, 2024 10:38
@jfrost-mo jfrost-mo merged commit cfc9303 into reduce_plot_resolution Aug 20, 2024
7 checks passed
@jfrost-mo jfrost-mo deleted the configurable_plotting_resolution branch August 20, 2024 10:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant