civicmapper / culvert-toolkit

GIS tools designed to support analysis of culvert performance within their watershed
https://civicmapper.github.io/culvert-toolkit/
Apache License 2.0
0 stars 0 forks source link

Move crossing-level analytics to the NaaccCrossing model #19

Open gassc opened 1 year ago

gassc commented 1 year ago

NaaccCrossing is currently just a container for grouping NaaccCulvert instances, but we could offload a lot of the business logic around determining max return period per-crossing, which is currently performaed in CulvertCapacity workflow methods, to the model (or an associated/coupled set of service functions).

This work would be a prerequisite for #16, because we'd be changing where iteration happens.