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

Mapillary Placement Tools #4426

Open
amahon opened this issue Oct 10, 2017 · 9 comments
Open

Mapillary Placement Tools #4426

amahon opened this issue Oct 10, 2017 · 9 comments
Labels
streetlevel An issue with streetlevel photos

Comments

@amahon
Copy link

amahon commented Oct 10, 2017

Building from SfM reconstructions that Mapillary generates everywhere we have imagery, it's possible to edit map data by placing and editing points in an image.

The pattern, deemed Placement Tools, is demonstrated by an example found in the Mapillary.JS documentation.

The functionality is expressed in a Mapillary add-in for ArcGIS Pro:
image

Currently, Mapillary.JS only supports placing and editing points in imagery. Forthcoming, but unscheduled, is support for linestrings and polygons.

Also, one caveat, Placement tools are only as good as Mapillary's underlying data is. In places like Amsterdam, where Mapillary has high quality imagery with very good GPS data, Placement Tools work fantastically. In areas where imagery/gps data is not as good, Placement Tools will not perform as well.

Let's think about how we can bring this functionality into iD Editor to allow OSM editors to place and edit nodes via Mapillary Imagery.

@amahon
Copy link
Author

amahon commented Oct 10, 2017

@bhousel - let me know what you think and how I can help!

@bhousel bhousel added the new-feature A new feature for iD label Oct 11, 2017
@bhousel
Copy link
Member

bhousel commented Oct 11, 2017

Very cool, this will make mapping objects so easy 👍

@planemad
Copy link
Contributor

Having this will completely unleash a new wave of detailed editing on OSM. I'm going to immediately use it for tree mapping when its out!🌲🌲🌲

@bhousel bhousel added streetlevel An issue with streetlevel photos and removed new-feature A new feature for iD labels Oct 25, 2017
@tordans
Copy link
Collaborator

tordans commented Dec 30, 2017

@amahon I saw a presentation about this (YouTube) and would love to see this feature live.
What is the status?
What is the timeline?
Who is writing the code to get it into iD?

Update: I tested the placement tool and ATM I don't think its working accurately enough to use it as part of the editor. See mapillary/mapillary-js#272.

@amahon
Copy link
Author

amahon commented Jan 4, 2018

@tordans - thanks for your questions!

Right now, we're working on implementing an API that will allow iD editor to access the detection data. I don't have an up to date timeline that I can share. Last discussed, which is a while ago now, Mapbox would implement the new features in iD -- since so much time has passed, I need to loop back and assess priorities with @bhousel. Will post an update here.

Let me know if I can help with anything else!

@tordans
Copy link
Collaborator

tordans commented Sep 29, 2018

@amahon any news about this?

@tordans
Copy link
Collaborator

tordans commented Feb 21, 2019

I noticed a surge of attention for this feature (outside of iD). For for reference, a few links:

@todrobbins
Copy link

todrobbins commented Feb 21, 2019

@amahon @cbeddow the people demand SfM tooling! 🛠

At least there's https://osm.cycle.travel/deriviste/ for now

@cbeddow
Copy link
Contributor

cbeddow commented Feb 25, 2019

Yes, Deriviste is a good example of how it can work. It also is in the go2mapillary plugin for QGIS, in case another example is needed. I wouldn't personally develop this for iD editor as Deriviste is way more simple for me to work with and OSM iD usually over my head, but if anybody is looking to add it I'm totally open to advise!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
streetlevel An issue with streetlevel photos
Projects
None yet
Development

No branches or pull requests

6 participants