Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get Mapillary Detections under a poweruser feature flag #1127

Open
Bonkles opened this issue Sep 25, 2023 · 3 comments
Open

Get Mapillary Detections under a poweruser feature flag #1127

Bonkles opened this issue Sep 25, 2023 · 3 comments
Assignees

Comments

@Bonkles
Copy link
Contributor

Bonkles commented Sep 25, 2023

Description

After speaking with Said about the object detections feature branch, we'll put it under a 'power user' feature and enable it in Rapid v2.

@Bonkles
Copy link
Contributor Author

Bonkles commented Oct 3, 2023

This work needs to be rebased on top of main, as it currently lives in the 2.0-derived branched called mly_objects_v2.

@Bonkles
Copy link
Contributor Author

Bonkles commented Oct 3, 2023

It would be a much nicer experience for the user if we could constrain the display of imagery points to ONLY those images that show a particular detection. This would greatly cut down on the visual clutter for mappers.

Bonus points: While an object is selected, the forward / backward < / > Buttons in the UI should go through the images that spot the object, not just advance the current road sequence. This would allow mappers to get a really quick contextual understanding of the feature and its alignment/correctness.

@Bonkles
Copy link
Contributor Author

Bonkles commented Oct 3, 2023

Conulted with @cbeddow about this, he did some research on the topic of 'only showing images that contain a detection', which would allow us to filter the images we display when an object is clicked.

According to his notes:

https://graph.mapillary.com/$MAP_FEATURE_ID?access_token=$TOKEN&fields=images

^ gives list of image IDs for that object, API Docs here: https://www.mapillary.com/developer/api-documentation#map-feature

Also:
https://graph.mapillary.com/$MAP_FEATURE_ID/detections?fields=value,image,geometry

For a map feature, this API call gives each image and detection geom, which I think is best displayed as a bbox of the detection in a bright color and not the unsimplified outline
^ right now I think we display too many detections (all of the outlines) and it would be better to get the bbox

@bhousel bhousel removed this from the v2.2.0 milestone Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants