Skip to content

Add fixes for vertical line xlim and PDF xlim#1182

Merged
jwarner8 merged 5 commits intomainfrom
use_json
Feb 24, 2025
Merged

Add fixes for vertical line xlim and PDF xlim#1182
jwarner8 merged 5 commits intomainfrom
use_json

Conversation

@jwarner8
Copy link
Contributor

@jwarner8 jwarner8 commented Feb 20, 2025

Also removes obslete references to the type of histogram (step filled, etc.) as this is no longer specified in the GUI and does not pass through into the recipes.

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.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2025

Coverage

@jwarner8 jwarner8 requested a review from jfrost-mo February 20, 2025 15:27
@jwarner8 jwarner8 self-assigned this Feb 20, 2025
@jfrost-mo jfrost-mo changed the title Add fixes for vertlcal line xlim and PDF xlim Add fixes for vertical line xlim and PDF xlim Feb 21, 2025
Copy link
Member

@jfrost-mo jfrost-mo left a comment

Choose a reason for hiding this comment

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

Looking good to me.

@jwarner8 jwarner8 merged commit 382b4bc into main Feb 24, 2025
9 checks passed
@jwarner8 jwarner8 deleted the use_json branch February 24, 2025 07:54
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