Skip to content

Fix phoenix filter handling#330

Merged
kujaku11 merged 22 commits into
masterfrom
fix_phoenix
Mar 24, 2026
Merged

Fix phoenix filter handling#330
kujaku11 merged 22 commits into
masterfrom
fix_phoenix

Conversation

@kujaku11

Copy link
Copy Markdown
Collaborator

Updated how filters are being handled for Phoenix MTU-5C files and workflow.

  • The main update was to rename the coil filter to what the recmeta.json would use
  • add logic to how the filters are handled when reading in a file.
  • add logic for keeping track of stage number such that the channel_response and channel_metadata.filters should be similar.

Generate changelog for unreleased changes with specific tag.
Add a new workflow (.github/workflows/bump-version.yml) that bumps the package version, generates changelog/HISTORY.rst, and commits changes when a release label (release:major/minor/patch) is added to an open PR.

Simplify .github/workflows/publish.yml to only run publish steps on PR close/merge, remove the prior in-publish version-bump/update job, and keep a single publish job that creates a GitHub Release and uploads to TestPyPI then PyPI. Also update the bump-my-version call to use `show current_version` and make minor permission/formatting adjustments.
Add a new example notebook (docs/examples/notebooks/new_documentation.ipynb) and update PhoenixClient to gracefully handle None inputs for receiver and calibration paths. The client now explicitly skips processing when value is None (avoiding unintended TypeError/ValueError), and includes a minor formatting cleanup for sensor calibration dict assignment.
@codecov

codecov Bot commented Mar 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.48718% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.13%. Comparing base (9f33ad4) to head (22a2a49).
⚠️ Report is 49 commits behind head on master.

Files with missing lines Patch % Lines
mth5/clients/phoenix.py 57.14% 9 Missing ⚠️
mth5/io/phoenix/readers/base.py 72.00% 7 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #330   +/-   ##
=======================================
  Coverage   81.12%   81.13%           
=======================================
  Files         232      232           
  Lines       47934    47966   +32     
=======================================
+ Hits        38887    38916   +29     
- Misses       9047     9050    +3     
Flag Coverage Δ
tests 81.13% <79.48%> (+<0.01%) ⬆️

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.

@kujaku11
kujaku11 merged commit f23f474 into master Mar 24, 2026
11 of 12 checks passed
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