Skip to content

Create a phys180E sub-module#176

Draft
rocco8773 wants to merge 15 commits intoBaPSF:masterfrom
rocco8773:add_180E
Draft

Create a phys180E sub-module#176
rocco8773 wants to merge 15 commits intoBaPSF:masterfrom
rocco8773:add_180E

Conversation

@rocco8773
Copy link
Member

This PR adds a phys180E sub-module to the bapsflib namespace for reading data from HDF5 files created in the Physics 180E class.

@codecov
Copy link

codecov bot commented Mar 10, 2026

Codecov Report

❌ Patch coverage is 17.18062% with 188 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.62%. Comparing base (d83cd04) to head (8f9ceeb).

Files with missing lines Patch % Lines
bapsflib/_hdf/maps/digitizers/lecroy.py 11.04% 145 Missing ⚠️
bapsflib/_hdf/maps/digitizers/map_digis.py 8.33% 33 Missing ⚠️
bapsflib/_hdf/utils/helpers.py 55.55% 2 Missing and 2 partials ⚠️
bapsflib/phys180E/_hdf/file_.py 50.00% 4 Missing ⚠️
bapsflib/_hdf/utils/hdfreaddata.py 60.00% 1 Missing and 1 partial ⚠️

❌ Your project check has failed because the head coverage (93.62%) is below the adjusted base coverage (98.58%). You can increase the head coverage or adjust the Removed Code Behavior.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
- Coverage   98.78%   93.62%   -5.17%     
==========================================
  Files          49       53       +4     
  Lines        3369     3590     +221     
  Branches      638      690      +52     
==========================================
+ Hits         3328     3361      +33     
- Misses         38      223     +185     
- Partials        3        6       +3     

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

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