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

Tracking software changes #23

Open
joto opened this Issue Mar 17, 2017 · 6 comments

Comments

Projects
None yet
5 participants
@joto
Member

joto commented Mar 17, 2017

There is, of course, a lot of software that handles or uses OSM (multi)polygons. This includes, but it not limited to, editors and renderers. Generally all that we are doing here is backwards compatible, so changes in those programs are not strictly necessary, but the goal is to make handling (multi)polygons easier for the software, so we want to tell them about the work here and also get any feedback they might have.

This (meta-)issue is for tracking issues on software that should be aware of the fixing efforts.

Editors

Data export

Libraries

  • libosmium

Other

Feel free to extend this issue with links to other projects that should be made aware of this effort.

@nebulon42

This comment has been minimized.

Show comment
Hide comment
@nebulon42

nebulon42 Mar 18, 2017

Contributor

Regarding Potlatch 2 I don't think this is only a minor issue. The software still creates old-style multipolygons. After my clean-up in Austria I noticed two cases where old-style multipolygons were re-introduced in changesets by P2. E.g. https://www.openstreetmap.org/changeset/46925424 or more specifically https://www.openstreetmap.org/relation/7080945 which was cleaned up already by another user.

Given the position of the software in its life-cycle I think it is unlikely that big changes will happen to it. I don't think this will affect the data on a large scale, but I'm not familiar with the usage stats of P2.

cc @systemed

Contributor

nebulon42 commented Mar 18, 2017

Regarding Potlatch 2 I don't think this is only a minor issue. The software still creates old-style multipolygons. After my clean-up in Austria I noticed two cases where old-style multipolygons were re-introduced in changesets by P2. E.g. https://www.openstreetmap.org/changeset/46925424 or more specifically https://www.openstreetmap.org/relation/7080945 which was cleaned up already by another user.

Given the position of the software in its life-cycle I think it is unlikely that big changes will happen to it. I don't think this will affect the data on a large scale, but I'm not familiar with the usage stats of P2.

cc @systemed

@joto

This comment has been minimized.

Show comment
Hide comment
@joto

joto Mar 18, 2017

Member

I talked with Richard yesterday about P2. He is aware of the problem and might fix this. I suspect, though, he doesn't want to change the behaviour now to support both old and new style. It might make sense to wait a bit and then just support the new (and at that time basically only) style. This suspect this will only be solved really once the main OSM map switches to new-style-only.

Member

joto commented Mar 18, 2017

I talked with Richard yesterday about P2. He is aware of the problem and might fix this. I suspect, though, he doesn't want to change the behaviour now to support both old and new style. It might make sense to wait a bit and then just support the new (and at that time basically only) style. This suspect this will only be solved really once the main OSM map switches to new-style-only.

@systemed

This comment has been minimized.

Show comment
Hide comment
@systemed

systemed Mar 18, 2017

Yep, pretty much that.

Yep, pretty much that.

@stoecker

This comment has been minimized.

Show comment
Hide comment
@stoecker

stoecker Mar 18, 2017

JOSM has a long-standing issue for this: https://josm.openstreetmap.de/ticket/10529 - We already have a expert parameter to disable old style stuff (I did not remember that I added it :-)

JOSM has a long-standing issue for this: https://josm.openstreetmap.de/ticket/10529 - We already have a expert parameter to disable old style stuff (I did not remember that I added it :-)

@ImreSamu

This comment has been minimized.

Show comment
Hide comment
@ImreSamu

ImreSamu Mar 20, 2017

Feel free to extend this issue with links to other projects

ArcGIS Editor for OSM: https://github.com/Esri/arcgis-osm-editor
* Search: outer way multipolygon

Feel free to extend this issue with links to other projects

ArcGIS Editor for OSM: https://github.com/Esri/arcgis-osm-editor
* Search: outer way multipolygon

@joto

This comment has been minimized.

Show comment
Hide comment
@joto

joto Mar 20, 2017

Member

Thanks @ImreSamu . I have opened an issue on their tracker.

Member

joto commented Mar 20, 2017

Thanks @ImreSamu . I have opened an issue on their tracker.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment