Skip to content

Fix/plot improvements#201

Merged
andre-merzky merged 10 commits intodevelfrom
fix/plot_improvements
May 13, 2025
Merged

Fix/plot improvements#201
andre-merzky merged 10 commits intodevelfrom
fix/plot_improvements

Conversation

@andre-merzky
Copy link
Member

support stdev plotting and plot height setting.

Copy link
Contributor

@mtitov mtitov left a comment

Choose a reason for hiding this comment

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

some small comments, otherwise looks good to go

Copy link
Contributor

@mtitov mtitov left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@codecov
Copy link

codecov bot commented Mar 31, 2025

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 23.11%. Comparing base (6ca50f5) to head (1ddc3e7).
Report is 19 commits behind head on devel.

Files with missing lines Patch % Lines
src/radical/analytics/utils/plot.py 20.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            devel     #201      +/-   ##
==========================================
- Coverage   23.15%   23.11%   -0.04%     
==========================================
  Files           7        7              
  Lines        1192     1194       +2     
==========================================
  Hits          276      276              
- Misses        916      918       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andre-merzky andre-merzky merged commit 1ab12e0 into devel May 13, 2025
3 of 4 checks passed
@andre-merzky andre-merzky deleted the fix/plot_improvements branch May 13, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants