Skip to content

Migrate all modules off docval + real signatures for generated classes (Phases 3-4 of #1129)#1527

Draft
bendichter wants to merge 5 commits into
typing-compat-layerfrom
typing-phase3-core
Draft

Migrate all modules off docval + real signatures for generated classes (Phases 3-4 of #1129)#1527
bendichter wants to merge 5 commits into
typing-compat-layerfrom
typing-phase3-core

Generate real type-hinted signatures for dynamic classes (Phase 4)

a255543
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Jul 4, 2026 in 0s

38.09% (-53.70%) compared to 7cb9cec

View this Pull Request on Codecov

38.09% (-53.70%) compared to 7cb9cec

Details

Codecov Report

❌ Patch coverage is 80.05391% with 148 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.09%. Comparing base (7cb9cec) to head (a255543).

Files with missing lines Patch % Lines
src/hdmf/common/table.py 65.27% 25 Missing ⚠️
src/hdmf/typing/_build.py 73.07% 11 Missing and 10 partials ⚠️
src/hdmf/container.py 66.66% 18 Missing ⚠️
src/hdmf/typing/migrate.py 0.00% 16 Missing ⚠️
src/hdmf/validate/errors.py 63.33% 11 Missing ⚠️
src/hdmf/backends/hdf5/h5_utils.py 58.33% 10 Missing ⚠️
src/hdmf/backends/hdf5/h5tools.py 84.78% 7 Missing ⚠️
src/hdmf/spec/spec.py 92.78% 7 Missing ⚠️
src/hdmf/term_set.py 68.75% 4 Missing and 1 partial ⚠️
src/hdmf/build/builders.py 86.66% 2 Missing and 2 partials ⚠️
... and 10 more

❗ There is a different number of reports uploaded between BASE (7cb9cec) and HEAD (a255543). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (7cb9cec) HEAD (a255543)
5 1
Additional details and impacted files 
@@                   Coverage Diff                    @@
##           typing-compat-layer    #1527       +/-   ##
========================================================
- Coverage                91.79%   38.09%   -53.70%     
========================================================
  Files                       50       51        +1     
  Lines                    11027    10952       -75     
  Branches                  2302     2337       +35     
========================================================
- Hits                     10122     4172     -5950     
- Misses                     558     6365     +5807     
- Partials                   347      415       +68

☔ View full report in Codecov by Harness.
📢 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.
View more details on Codecov