Skip to content

MapCSS rules for France (from JOSM) #497

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

Closed
wants to merge 1 commit into from
Closed

MapCSS rules for France (from JOSM) #497

wants to merge 1 commit into from

Conversation

PanierAvide
Copy link
Contributor

So I used the tool to convert from the https://josm.openstreetmap.de/wiki/Rules/FranceSpecificRules MapCSS file, and used item 9016 for this gauge on railway rule.

frodrigo added a commit to frodrigo/osmose-backend that referenced this pull request May 2, 2019
@frodrigo
Copy link
Contributor

frodrigo commented May 2, 2019

I'm impressed, you build a mapcss files!

Unfortunately most steps was necessary. I update the mapcss source file to add -osmose attributes. Then rename osmose assert attributes, and finally regenerated the python file and changing generator configuration.

@PanierAvide
Copy link
Contributor Author

Great, thanks for having done the last steps, that makes clearer for me what is necessary to have a complete MapCSS analyzer for Osmose :-)

@PanierAvide PanierAvide closed this May 3, 2019
jocelynj added a commit that referenced this pull request May 4, 2019
* frodrigo/master:
  Fix downloader in case of excaption
  Fix tests on action table
  Stop psql script on error
  Allow 'snc' as housenumber in Italy into TagFix_Housenumber
  py3: analyser_merge_restaurant_FR_aquitaine
  py3: fix Analyser_Merge
  Update opendata analyser_merge_bicycle_parking_FR_bordeaux
  Switch data format to geojson in analyser_merge_bicycle_rental_FR_IDF
  Add mapccs plugin Josm_FranceSpecificRules #497
  mapcss: rename assertMatchWithContext to -osmoseAssertMatchWithContext
  Fix analyser_merge_school_FR for py2
@frodrigo
Copy link
Contributor

frodrigo commented May 6, 2019

@PanierAvide
Copy link
Contributor Author

Yes I saw that this morning, that's great :-) Thanks for the fix and integration !

@frodrigo
Copy link
Contributor

frodrigo commented May 8, 2019

The new item is now publicly available.

@didier2020
Copy link
Contributor

I made a modification to https://josm.openstreetmap.de/wiki/Rules/FranceSpecificRules :
remove "ref" in nodes'rule highway milestone.
Can you do an osmose update for this mapcss rule ?

@frodrigo
Copy link
Contributor

frodrigo commented Jun 8, 2020

@didier2020 maybe wait a bit more as we are agree?

@didier2020
Copy link
Contributor

yes i can...
those who think that ref should not be a tag, do not offer an alternative

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

Successfully merging this pull request may close these issues.

3 participants