Skip to content

Consolidate cartesian interp option and introduce structured SimulationResult return - #647

Open
faridyagubbayli wants to merge 6 commits into
masterfrom
chore/minor-code-improvements
Open

Consolidate cartesian interp option and introduce structured SimulationResult return#647
faridyagubbayli wants to merge 6 commits into
masterfrom
chore/minor-code-improvements

SimulationResult: align field names with sensor.record (drop _raw suf…

45967ab
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 26, 2026 in 0s

76.57% (+1.50%) compared to cf4ec8f

View this Pull Request on Codecov

76.57% (+1.50%) compared to cf4ec8f

Details

Codecov Report

❌ Patch coverage is 93.61702% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.57%. Comparing base (cf4ec8f) to head (45967ab).
⚠️ Report is 58 commits behind head on master.

Files with missing lines Patch % Lines
kwave/data.py 91.89% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #647      +/-   ##
==========================================
+ Coverage   75.06%   76.57%   +1.50%     
==========================================
  Files          57       57              
  Lines        8126     8686     +560     
  Branches     1582     1804     +222     
==========================================
+ Hits         6100     6651     +551     
+ Misses       1405     1398       -7     
- Partials      621      637      +16     

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