Skip to content

feature/bokeh and bugfix/tab-readers#31

Merged
c-randall merged 12 commits into
mainfrom
feature/bokeh
May 18, 2026
Merged

feature/bokeh and bugfix/tab-readers#31
c-randall merged 12 commits into
mainfrom
feature/bokeh

Conversation

@c-randall

@c-randall c-randall commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Description

Adds a new interactive plot method using bokeh as a backend. Also fixes a read error with datasets from ANL that have extra tabs by ignoring ragged columns.

Some additional aliases were also added to catch Cap. [Ah] and Ener. [Wh].

Type of change

  • New feature (non-breaking change which adds functionality)
  • Optimization (back-end change that improves speed/readability/etc.)
  • Bug fix (non-breaking change which fixes an issue)
  • Chore (any other clean up, maintenance, etc.)

Key checklist:

  • No style issues: $ nox -s linter [-- format]
  • Code is free of misspellings: $ nox -s codespell [-- write]
  • All tests pass: $ nox -s tests

Further checks:

  • The documentation builds: $ nox -s docs.
  • Code is commented, particularly in hard-to-understand areas.
  • Tests are added that prove fix is effective or that feature works.

@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.83562% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.31%. Comparing base (beadf35) to head (f4918bd).

Files with missing lines Patch % Lines
src/ampworks/plotutils/_bokeh.py 86.56% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   60.26%   63.31%   +3.04%     
==========================================
  Files          43       44       +1     
  Lines        2001     2107     +106     
==========================================
+ Hits         1206     1334     +128     
+ Misses        795      773      -22     
Flag Coverage Δ
unittests 63.31% <93.83%> (+3.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@c-randall
c-randall marked this pull request as ready for review May 18, 2026 17:56
@c-randall
c-randall merged commit 9dfcb52 into main May 18, 2026
23 checks passed
@c-randall
c-randall deleted the feature/bokeh branch May 18, 2026 17:56
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.

1 participant