sharedstreets / curb-wheel

Curb mapping wheel (currently in-progress)
MIT License
6 stars 0 forks source link

Oct 26 punchlist (targets for alpha version so we can go map) #1

Open russbiggs opened 4 years ago

russbiggs commented 4 years ago

Needed before we can map SF and release to testing cohort:

emilyeros commented 4 years ago

Some bugs I'm seeing:

  1. Digitizer can't recall images once a regulation is digitized. The moment there's a template name, the images disappear for that reg and don't come back unless I wipe the regulation data.

  2. Digitizer won't export the feed under some circumstances. The survey that was breaking the digitizer contained a fire hydrant, overly long streets, and overly short streets. I've tested long and short streets and those didn't cause the error. Console message: image

  3. Fire hydrants aren't getting their shstLocationStart and shstLocationEnd properties updated. These should reflect the transformed length/location of the regulation. image

3b. Relatedly, segments on overly-long streets aren't having their shstLocationStart and shshLocationEnd properties updated, or their geometries scaled back. This feature should take up 50% of the street, but it takes up the whole thing. (Feature was mapped at 71/71m, then I rolled to 140m and saved it) image

3c. Segments on overly short streets also aren't getting centered. This one should be shifted closer to the center of the street but it appears at the edge: image

  1. Features that don't have a digitized regulation should be omitted from the exported CurbLR feed.

  2. Minor point: Some non-populated fields are writing out as null when they should either be excluded entirely or else they should appear as an empty array. Example: see userClasses below image

emilyeros commented 4 years ago

Update: I can't isolate the event that caused my export to fail. I tried:

I'm not sure what triggered the fail or if it was a combination of factors. Survey tested: 2020-11-17T00:30:46.234Z/