Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Bug: Fix shape assignment for DISU package#2651

Merged
wpbonelli merged 2 commits into
modflowpy:developfrom
maseology:patch-1
Nov 18, 2025
Merged

Bug: Fix shape assignment for DISU package#2651
wpbonelli merged 2 commits into
modflowpy:developfrom
maseology:patch-1

Conversation

@maseology

Copy link
Copy Markdown
Contributor

changing the int to a list of int avoids the TypeError: 'int' object is not iterable

changing the int to a list of int avoids the TypeError: 'int' object is not iterable
@wpbonelli

Copy link
Copy Markdown
Member

thanks @maseology I guess ruff prefers (nodes,)?

@codecov

codecov Bot commented Nov 17, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 71.4%. Comparing base (556c088) to head (9bb2f51).
⚠️ Report is 86 commits behind head on develop.

Files with missing lines Patch % Lines
flopy/modpath/mp7.py 0.0% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2651      +/-   ##
===========================================
+ Coverage     55.5%    71.4%   +15.9%     
===========================================
  Files          644      667      +23     
  Lines       124135   129332    +5197     
===========================================
+ Hits         68947    92423   +23476     
+ Misses       55188    36909   -18279     
Files with missing lines Coverage Δ
flopy/modpath/mp7.py 86.5% <0.0%> (+<0.1%) ⬆️

... and 559 files with indirect coverage changes

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

@maseology

Copy link
Copy Markdown
Contributor Author

thanks @maseology I guess ruff prefers (nodes,)?

Yeah, and not that it gets you that far, this only avoided the cryptic error message. Thanks.

@wpbonelli wpbonelli merged commit 44c9071 into modflowpy:develop Nov 18, 2025
23 of 25 checks passed
@maseology maseology deleted the patch-1 branch January 11, 2026 13:09
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