Riverscapes / RaveAddIn

RAVE AddIn for ArcGIS
http://rave.riverscapes.xyz/
GNU General Public License v3.0
1 stars 3 forks source link

Missing Confinement Values #91

Closed philipbaileynar closed 3 years ago

philipbaileynar commented 3 years ago

I think this is a problem with the confinement symbology, not the confinement model itself. I ran 17060304 and loaded the confinement margins in ArcMap. Noticed a reach that was absent from the map. This typically means that it was omitted from the layer symbology.

Case Confinement Ratio Constriction Ratio
Absent Reach 1 10
Green Reaches 0 0
Screen Shot 2020-10-28 at 2 36 49 PM

Please fix and re-upload relevant layer files.

joewheaton commented 3 years ago

This is really exciting. Lets talk about terminology, business logic and symbology when you're ready.


Dr. Joseph M. Wheaton 435-554-1247 (direct) | http://www.joewheaton.org

On Wed, Oct 28, 2020 at 3:41 PM Philip Bailey notifications@github.com wrote:

I think this is a problem with the confinement symbology, not the confinement model itself. I ran 17060304 and loaded the confinement margins in ArcMap. Noticed a reach that was absent from the map. This typically means that it was omitted from the layer symbology. Case Confinement Ratio Constriction Ratio Absent Reach 1 10 Green Reaches 0 0

[image: Screen Shot 2020-10-28 at 2 36 49 PM] https://user-images.githubusercontent.com/8645379/97499488-2893e000-192b-11eb-91d9-fe5a1461d0e4.png

Please fix and re-upload relevant layer files.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Riverscapes/RaveAddIn/issues/91, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACKGAAM22YD7ZLREZG5XRO3SNCFZNANCNFSM4TC4353Q .

philipbaileynar commented 3 years ago

@KellyMWhitehead did we ever figure this out?

KellyMWhitehead commented 3 years ago

Issue was related to geometry tolerance when splitting the confinement buffer by the centerline. Was able to extend the splitting lines by using "scale" and seems to work for all polygons so far.

Fixed: 18a0ba0a2cdb15a98dde60aa3c3bb2f9a312ca1d