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.

Sign up for GitHub

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

Jump to bottom

construction=minor is not an error #545

Closed
joostschouppe opened this issue Jun 28, 2019 · 2 comments
Closed

construction=minor is not an error #545

joostschouppe opened this issue Jun 28, 2019 · 2 comments
Labels
bug ready

Comments

@joostschouppe
Copy link

I have the impression Osmose returns all highways with construction=minor as an error. However, this is used by mappers to indicate "roads with some construction works, but not closed due to construction". There are 6000 cases of this and it is documented on the construction=* wiki page
@frodrigo
Copy link
Member

frodrigo commented Jul 1, 2019

You are right, this case is not handled, yet.

@frodrigo frodrigo added the bug label Jul 1, 2019
frodrigo added a commit to frodrigo/osmose-backend that referenced this issue Jul 1, 2019
@frodrigo
Handle construction=minor in Construction2.validator.mapcss osm-fr#545
774f8a8
@frodrigo frodrigo added the ready label Jul 1, 2019
frodrigo added a commit to frodrigo/osmose-backend that referenced this issue Jul 5, 2019
@frodrigo
Handle construction=minor in Construction2.validator.mapcss osm-fr#545
01e5922
frodrigo added a commit to frodrigo/osmose-backend that referenced this issue Jul 8, 2019
@frodrigo
Handle construction=minor in Construction2.validator.mapcss osm-fr#545
ab040fd
frodrigo added a commit to frodrigo/osmose-backend that referenced this issue Jul 8, 2019
@frodrigo
Handle construction=minor in Construction2.validator.mapcss osm-fr#545
0afdf80
jocelynj added a commit that referenced this issue Jul 8, 2019
@jocelynj
Merge remote-tracking branch 'frodrigo/master'
d47fadd 
* frodrigo/master:
  Use name:left and name:right & better relation location in analyser_osmosis_relation_associatedStreet #551
  White list amenity=stables + leisure=horse_riding in TagRemove_Incompatibles #553
  Active Phone plugin for Cuba
  Fix Phone for Spain #546
  Update plugin Phone Spain
  Add club as main tag in TagFix_MultipleTag #550
  Handle construction=minor in Construction2.validator.mapcss #545
  Remove hospital/doctors in shop_FR.mapping.json #338
  In analyser_merge_shop_FR also filter on trancheEffectifs #338
  shop_FR.mapping.json replace note=FIXME by fixme= #338
  Exclude Appartements and "chez" in analyser_merge_shop_FR #338
  Revert change on ST_DistanceSphere analyser_osmosis_water #448
  mapcss: regen from mapcss
  mapcss: fix test code generation
  mapcss: skip area selector
@frodrigo
Copy link
Member

frodrigo commented Jul 9, 2019

Done.

@frodrigo frodrigo closed this as completed Jul 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug ready
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@frodrigo @joostschouppe