NOAA-OWP / hydrovis

Other
11 stars 8 forks source link

AEP FIM: RAS2FIM Bugs #879

Open shawncrawley opened 6 days ago

shawncrawley commented 6 days ago

An issue was reported to us indicating that the RAS2FIM polygons were missing from the static_nwm_aep_inundation_extent_library_noaa service in UAT. While investigating, we confirmed the issue, but also uncovered a few other issues. Namely:

Update: Sep 16, 2024


Safety test: To test, find a ras2fim feature ID, then filter a layer for that feature id. If it is a ras2fim record, you should only get either 0 or 1 feature returned. If 0, it was dropped for some reason, maybe flow ranges or something, but worth more researching. If more than 1, something is very wrong. ras2fim can only have one aep feature per aep stage. But HAND will have 1 to many as it has multiple hydroid (features) crosswalked to one feature ID.

shawncrawley commented 2 days ago

We actually found that the issue with RAS2FIM not drawing was actually stemming from the polygons being ingested into the ras2fim.geocurves table with the wrong projection. So technically RAS2FIM is drawing, just in the completely wrong place: in the sea far north of Russia. We have corrected the projection bug in our ingest scripts and are re-running those now.

RobHanna-NOAA commented 2 days ago

The ras2fim data load was incorrectly applying the wrong crs. I thought incoming it was 5070 but it is really 2277. Now that we changed the incoming ras2fim csv's to 2277 and reprojected it in the data load scripts to 3857, we should be good. We did a quick test after loading one ras2fim csv and it was good now. Re-running the full ras2fim as we speak.

RobHanna-NOAA commented 1 day ago

Fixed the crs problem. Re-ran the ras2fim data, checked just that data in arc and the mapviewer (??). Now re-running AEP and hopefully we are all good.

RobHanna-NOAA commented 18 hours ago

All fixed now for the ras2fim.geocurves and mas_geocurves.