Skip to content

Fix(test_save_load_node_mapping_structured): - #2838

Merged
jlarsen-usgs merged 2 commits into
modflowpy:developfrom
jlarsen-usgs:develop
Sep 3, 2026
Merged

Fix(test_save_load_node_mapping_structured): #2838
jlarsen-usgs merged 2 commits into
modflowpy:developfrom
jlarsen-usgs:develop

Conversation

@jlarsen-usgs

@jlarsen-usgs jlarsen-usgs commented Sep 3, 2026

Copy link
Copy Markdown
Contributor
  • update _node_map to _node_map_arr to follow new convention

  • remove sklearn requirement as test was getting skipped and now sklearn is not required

…map_arr to follow new convention

* remove sklearn requirement
@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.5%. Comparing base (556c088) to head (9b24dfe).
⚠️ Report is 223 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2838      +/-   ##
===========================================
+ Coverage     55.5%    73.5%   +18.0%     
===========================================
  Files          644      659      +15     
  Lines       124135   132222    +8087     
===========================================
+ Hits         68947    97301   +28354     
+ Misses       55188    34921   -20267     

see 586 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jlarsen-usgs
jlarsen-usgs merged commit b59605b into modflowpy:develop Sep 3, 2026
23 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