Skip to content

FIX: Update violin plot to use new parameter orientation. #925

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
May 21, 2025
Merged

FIX: Update violin plot to use new parameter orientation. #925

merged 2 commits into from
May 21, 2025

Conversation

zssherman
Copy link
Collaborator

@zssherman zssherman commented May 13, 2025 •
edited
Loading

vert is deprecated, need to use orientation now.

  • Tests updated
  • Documentation reflects changes
  • PEP8 Standards or use of linter

@zssherman
FIX: Update violin plot to use new paramter orientation.
597da5c 
vert is deprecated, need to use orientation now.
@zssherman zssherman closed this May 13, 2025
@zssherman zssherman reopened this May 13, 2025
@zssherman
Copy link
Collaborator Author

@mgrover1 When you have time, this one should be good to go as well.

@AdamTheisen AdamTheisen merged commit 956b7a6 into ARM-DOE:main May 21, 2025
22 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AdamTheisen AdamTheisen AdamTheisen approved these changes

@mgrover1 mgrover1 Awaiting requested review from mgrover1 mgrover1 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zssherman @AdamTheisen