Skip to content

chore: codecov path fix, author name, NWB coverage, docs clarification#27

Merged
anzalks merged 6 commits into
mainfrom
codecov_path_resolution_nwb_export_issue
Jun 13, 2026
Merged

chore: codecov path fix, author name, NWB coverage, docs clarification#27
anzalks merged 6 commits into
mainfrom
codecov_path_resolution_nwb_export_issue

Conversation

@anzalks

@anzalks anzalks commented Jun 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • Remove relative_files = true (root cause of Codecov 0-files-matched)
  • Fix [tool.coverage.paths] source to map from src not src/Synaptipy
  • Remove stale rm -f .coverage step from test.yml
  • Remove */exporters/nwb_exporter.py from coverage omit (enforce coverage)
  • Fix author name: Anzal K Shahul → Anzal K. Shahul
  • Clarify graphics engine description in docs/index.rst (CPU-vectorized, not GPU-accelerated)
  • Add empirical validation benchmark script and reference CSV

No version bump. No PyPI release triggered.

anzalks added 6 commits June 1, 2026 23:14
… unit validation to core analysis, and update ABF import error handling.
This commit introduces a new script, `benchmark_real_data.py`, which performs empirical validation of SynaptiPy metric extractions against legacy ground-truth measurements from Clampfit and Stimfit. The script generates a scatter plot to visualize the correlation between the two measurement methods and saves the output plot for documentation purposes. It also includes error handling for missing files and computes Pearson correlation metrics.
@anzalks anzalks merged commit 5779fd8 into main Jun 13, 2026
26 checks passed
@anzalks anzalks deleted the codecov_path_resolution_nwb_export_issue branch June 13, 2026 08:26
@github-actions

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/Synaptipy/application
  session_manager.py
  src/Synaptipy/core/analysis
  passive_properties.py 352-356, 759-763
  synaptic_events.py
  src/Synaptipy/infrastructure/exporters
  nwb_exporter.py 338, 343, 345, 347, 351, 359
  src/Synaptipy/infrastructure/file_readers
  neo_adapter.py 537-538, 540-541
Project Total  

This report was generated by python-coverage-comment-action

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 55.26316% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.55%. Comparing base (a93f4bb) to head (207df55).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
src/Synaptipy/core/analysis/passive_properties.py 28.57% 10 Missing ⚠️
...Synaptipy/infrastructure/exporters/nwb_exporter.py 58.33% 5 Missing ⚠️
...naptipy/infrastructure/file_readers/neo_adapter.py 75.00% 2 Missing ⚠️

❌ Your patch status has failed because the patch coverage (55.26%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff            @@
##           main      #27       +/-   ##
=========================================
+ Coverage      0   91.55%   +91.55%     
=========================================
  Files         0       55       +55     
  Lines         0     7826     +7826     
=========================================
+ Hits          0     7165     +7165     
- Misses        0      661      +661     
Flag Coverage Δ
unittests 91.55% <55.26%> (?)

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

Files with missing lines Coverage Δ
src/Synaptipy/application/session_manager.py 100.00% <100.00%> (ø)
src/Synaptipy/core/analysis/synaptic_events.py 95.76% <100.00%> (ø)
src/Synaptipy/templates/plugin_template.py 100.00% <ø> (ø)
...naptipy/infrastructure/file_readers/neo_adapter.py 87.55% <75.00%> (ø)
...Synaptipy/infrastructure/exporters/nwb_exporter.py 48.19% <58.33%> (ø)
src/Synaptipy/core/analysis/passive_properties.py 96.70% <28.57%> (ø)

... and 49 files with indirect coverage changes

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

2 participants