Skip to content

Add river-constrained D8 flow direction - #1

Draft
kongdd wants to merge 28 commits into
mainfrom
feature/river-constrained-flowdir
Draft

Add river-constrained D8 flow direction#1
kongdd wants to merge 28 commits into
mainfrom
feature/river-constrained-flowdir

Run flow direction validation tests

d3cefde
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 21, 2026 in 0s

87.23% (+4.98%) compared to 8dab4e5

View this Pull Request on Codecov

87.23% (+4.98%) compared to 8dab4e5

Details

Codecov Report

❌ Patch coverage is 92.61364% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.23%. Comparing base (8dab4e5) to head (d3cefde).

Files with missing lines Patch % Lines
src/constrained_flowdir.jl 92.97% 22 Missing ⚠️
src/flowdir_qc.jl 71.42% 10 Missing ⚠️
ext/RiverGraphsArchGDALExt.jl 84.61% 6 Missing ⚠️
src/priority_flood.jl 99.05% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #1      +/-   ##
==========================================
+ Coverage   82.24%   87.23%   +4.98%     
==========================================
  Files          13       18       +5     
  Lines         569     1097     +528     
==========================================
+ Hits          468      957     +489     
- Misses        101      140      +39     

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