search

kcigeospatial / MDOT-SHA-NPDES-Next-Gen

Code and issues related to the MDOT SHA NPDES Project. Project codes: Config = 31, Management = 32.
0 stars 0 forks source link

Add Image Guide to Survey123 #293

Open talllguy opened 5 years ago

talllguy commented 5 years ago

Following on to #74, I'm adding the new image guide per @KCI-Ablowers and @brentreeves75

things to update on form

talllguy commented 5 years ago

@brentreeves75 I compared the order of the items in the image guide to that of the current inspection. They do not match. Would you like to change the order of the Survey123 form to match that of the guide? Here's a comparison:

Image Guide Order BMP Inspection Order
1 Debris Visibility
2 Inflow condition Mowability
3 Inflow stability BMP Access
4 BMP Vegetation Debris
5 BMP Contamination Fencing
6 Ponding Public Hazard
7 Excessive Pond Depth Potential for Public Hazard
8 Permanent Pool BMP Vegetation
9 Pretreatment Site Vegetation
10 Mowability BMP Contamination
11 Access Pre-Treatment
12 (skipped) Inflow Stability
13 Conveyance Stability Inflow Condition
14 Downstream Channel Condition Downstream Condition
15 Site Vegetation Conveyance Stability
16 Upstream Cover Upstream Cover
17 Upstream Embankment Stability Upstream Embankment Erosion
18 Upstream Embankment Toe Upstream Embankment Toe
19 Downstream Cover Downstream Cover
20 Downstream Embankment Stability Downstream Embankment Erosion
21 (skipped) Downstream Embankment Toe
22 (skipped) Embankment Seepage
23 Emergency Spillway Stability Emergency Spillway Opening
24 (skipped) Emergency Spillway Stability
25 Orifice opening Safe Water Depth
26 Orifice Trash Rack Ponding
27 (skipped) Permanent Pooling
28 Riser Trash Rack Orifice Opening
29 Riser Sediment Orifice Trash Rack
30 Riser Structure Control Structure
31 (skipped) Control Structure Opening
32 Principle Spillway Control Structure Sediment
33 Spillway Outfall Control Structure Trash Rack
34 (skipped) Control Structure Valve
35 (skipped) Spillway Outfall
36 Fencing Principle Spillway
KCI-Ablowers commented 5 years ago

My vote would be not to change the order. I was finally remembering where stuff was out in the field.

talllguy commented 5 years ago

If not the inspection form then what about the image guide?

talllguy commented 5 years ago

Also, assuming "Excessive Pond Depth" is the same as "Safe Water Depth" SAFE_WATERDEPTH in the form.

talllguy commented 5 years ago

What the guide calls Downstream Channel Condition is just Downstream Condition in our tool

talllguy commented 5 years ago

Assuming Upstream Embankment Stability is the same as Upstream Embankment Erosion in the form

KCI-Ablowers commented 5 years ago

If not the inspection form then what about the image guide?

I think for now we don't need to worry about it. Since its broken up people probably wont notice the numbers on the images.

talllguy commented 5 years ago

The KCI_1 form is updated to v3.3.0 with the new Image Guide

talllguy commented 5 years ago

Welp, the new version crashes both iPad's and S123 on windows. We'll need to rethink this approach.

KCI-Ablowers commented 5 years ago

I think we should try posting the 2 pages per item and putting a hyperlink in instead of the photos. This could be a good fall back.

talllguy commented 5 years ago

pushed test v3.3.1 which strips out all but Mowability while leaving all the media in the folder. Appears successful on the iPad.

KCI-Ablowers commented 5 years ago

pushed test v3.3.1 which strips out all but Mowability while leaving all the media in the folder. Appears successful on the iPad.

Confirmed it works on the mowability rating.

talllguy commented 5 years ago

There are many variables at play to debug. In the meantime, there is a PDF link to the image guide at the top of the form when the guide is enabled. Let me reach out to Esri to see if there is some undocumented limitation at play.

KCI-Ablowers commented 5 years ago

Sounds good. When i do the demo Ill point out the link and show the mowability as a "proof of concept"

talllguy commented 5 years ago

Note to self: summarize differences for KL

talllguy commented 5 years ago

I performed iterative testing to deduce the issue here.

Tests and Results

  1. Group for each IG and two pages within at high resolution ❌
  2. Remove group for each IG and same pages at hi res ❌
  3. Remove all groups but one (mowability) ✅
  4. Leave all groups, combine two pages and reduce to 600 px, debris only ✅

Analysis

Test 4 lets us know that the large number of groups is not at issue. It must be the file size or possibly height of all images. In test 3, I combine the pages and reduced the size. I'll do that for all of them and I think it should be fine.

talllguy commented 5 years ago

Also, I renamed the label to [whatever] Image Guide and put a green bar at the top so the name shows up in Survey123 (the note name is displayed at the top of the app when you tap a photo).