facebook / Rapid

The OpenStreetMap editor driven by open data, AI, and supercharged features
https://rapideditor.org
ISC License
505 stars 92 forks source link

Make Mapillary detections actually selectable #1517

Closed bhousel closed 2 weeks ago

bhousel commented 3 weeks ago

We have a lot of old issues related to working with the Mapillary detections.

For context, viewing these detections on the map as markers (detected objects and street signs) is a feature that has been around since the iD days, but actually working with these detections is clumsy.

1502 - Detected objects in the Mapillary viewer have lost their styling

1113 - Show which images contain a particular detection (I am highlighting them yellow now)

944 - Make Mapillary Objects addable (maybe also #1127?)

897 - Missing Label Popups (the user can not see what the detected object actually is)

686 - Collect 👍 / 👎 feedback on Mapillary detections

528 / #1024 - The old "Add Mapillary Features" experiment that we lost during the V2 rewrite

As a first step towards closing out some of this old work, today I made the detections actually selectable.

This means - these detections finally work like all the other selectable things in Rapid (features, QA Items, Rapid stuff, custom data):

select mode for detections

Followup ideas

bhousel commented 2 weeks ago

followup to above

The object icons are pretty bad, I will clean them up another time. We could really just map each of these detection categories to an OSM key/value and use the existing PresetSystem to pick reasonable icons. (Replacing the icons would also mean one less spritesheet we need to load)

☝️ I did this.. I mapped each detection type to a reasonable preset ID and now we can just use preset icons for these markers.

This is helpful too because if we ever do decide to allow mappers to add these features to OSM, it's easy to just create a point and assign a preset to it - all the tags will be set correctly.

For now I'm displaying them in the same "Mapillary green" color as the photo markers.
(@Bonkles said this looks very "Battlezone" haha)

old

Screenshot 2024-08-23 at 3 16 47 PM

new

Screenshot 2024-08-23 at 3 07 56 PM