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

Support multiple layers for managing imports #2586

Open
mikelmaron opened this issue Apr 12, 2015 · 8 comments
Open

Support multiple layers for managing imports #2586

mikelmaron opened this issue Apr 12, 2015 · 8 comments
Labels
new-feature A new feature for iD

Comments

@mikelmaron
Copy link

JOSM allows multiple data layers, which is handy when doing a merge of import data. Makes it easy to compare vs existing OSM data, and you can copy/paste features or tags between layers. For iD, simply allowing to have an OSM layer, and a "reference" layer (perhaps just a GeoJSON file passed by URL or from the local computer), could suffice.

@bhousel
Copy link
Member

bhousel commented Apr 12, 2015

As an intermediate step towards this, I think it makes sense to give the GPX layer a better name (suggestions welcome) and support a few other file formats. I'm kind of surprised iD doesn't support geojson here.

Then, actually giving the user some ability to import those features as OSM objects would be cool too, if we can keep it simple and do it in a way that doesn't break anything.

@mikelmaron
Copy link
Author

@bhousel makes sense. first step sounds pretty straightforward? and second step, what would it take to make the "(suggestions welcome)" layer interactive, clickable, and copyable?

@bhousel
Copy link
Member

bhousel commented Apr 14, 2015

what would it take to make the "(suggestions welcome)" layer interactive, clickable, and copyable?

I don't think that we would need to make the reference layer editable.. We could just add a button to import whatever is there (gpx/kml/geojson) onto the main map where it will be editable. Some potential issues:

  • Imported features need to have presets assigned
  • Imported features often need simplification and we don't have an operation for that (yet)
  • Ensure that the user understands not to import improperly licensed data
  • Consensus that this is a feature that should be built in the first place. OSM community tends to be pretty anti-import lately.

But anyway I like the idea. I have a bunch of gpx tracks sitting on my computer that I am too lazy to trace.

@pnorman
Copy link
Contributor

pnorman commented Apr 14, 2015

Is this in scope for a basic editor? I recall imports an external data sources being out of I'd scope in the past.

@ajturner
Copy link

ajturner commented Jun 8, 2015

I think that this would be very helpful - at least from the concept story:

"As a User, when editing in iD, I can add layers from other Services that I can edit and push into OpenStreetMap"

  • iD should keep it's core data model focused on OSM nodes/ways/relations.
  • a plugin would be loaded at runtime that adds support for API conversion
  • a URL to a GeoJSON file would be handled by this plugin
  • plugin parses the GeoJSON and converts it into OSM nodes/ways and hands it to iD core - output is same as if the user had hand-drawn the objects
  • User can now interact with the objects and commit to OSM

This is something I'd like to help out on.

@systemed
Copy link
Contributor

This has long been a feature of P2 - you can load various file formats (.shp, .gpx, .kml, .osm) which appear as "background layers", and are invisibly converted into ways and nodes on import. As background layers they don't do anything, but alt-clicking a feature in one of them (in traditional Potlatch obscure-keypress fashion) brings it through to the main editable layer as a live feature.

@slibby
Copy link
Contributor

slibby commented Jul 25, 2017

We have opened a new issue to discuss this topic for adding web-service based vector data from ArcGIS GeoServices (ArcGIS Server Map Services and Feature Services) as background + editable layers: #4164. I would like to think that the new issue gets at some of these requests and is similar to the function of uploading a shapefile or geojson document to iD and using that as a base layer. Adding shapefile and GeoJSON support would be a straightforward addition to the work we have done so far if it was of interest.

@bhousel
Copy link
Member

bhousel commented Jul 25, 2017

Awesome @slibby - yes I am also seeing #4164 as a great starting point towards improving how imports are done and would supersede this issue.

I have to comment on that thread, but was holding off to let everyone have their say and also because I've been focused on getting 2.3.2 released. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature A new feature for iD
Projects
None yet
Development

No branches or pull requests

6 participants