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

Add rendering for leisure=water_park #370

Closed
PauloCarvalhoRJ opened this issue Feb 28, 2014 · 31 comments
Closed

Add rendering for leisure=water_park #370

PauloCarvalhoRJ opened this issue Feb 28, 2014 · 31 comments

Comments

@PauloCarvalhoRJ
Copy link

Map: http://www.openstreetmap.org/#map=17/-22.98075/-43.48637

To the south of the parking lot there is a water park, represented by a polygon (id=263566033) with leisure=water_park and name=Rio Water Planet. There is also a node (id=2692030832) with the same tags. None of them are being rendered in OSM slippy map. If you enable "Map data" you can see its outline.

I've already checked for cache issues, etc. The parking lot and the neighbouring club belong to the same changeset and both are being rendered, thus this is not a cache issue.

Thanks.

@PauloCarvalhoRJ
Copy link
Author

Anyone?

@mrwojo
Copy link
Contributor

mrwojo commented Mar 7, 2014

We can add this to the theme park layer.

@PauloCarvalhoRJ
Copy link
Author

No solution to this yet? Is it complicated?

@gravitystorm
Copy link
Owner

@PauloCarvalhoRJ patience, please - it's not a high priority and there are literally hundreds of other things to work on too.

@PauloCarvalhoRJ
Copy link
Author

I do have patience (13 days, 2 posts). Furthermore I can do it myself if you point out how.

@Rovastar
Copy link
Contributor

Patience is relative. On in CartoCSS 13 days is not long at all.

Fell free to install what is needed for cartoCSS and produce a pull request.

@pnorman
Copy link
Collaborator

pnorman commented Mar 14, 2014

We're currently in the easier to wrangle stage of development where we're cleaning up the style still. New feature requests are currently not a development priority. A pull request won't significantly change matters.

@matthijsmelissen
Copy link
Collaborator

@daganzdaanda
Copy link

Hi, I just want to vote this up, too ;) It's getting summer, and people are starting to go to the swimming pools.
A water_park could maybe render in a lighter version of the sport_centre green. I guess many water_parks will have details like swimming pools, meadows, footways, buildings, parking, waterslides, beachvolleyball or other pitches. So it should look good also with these details.

@kaldari
Copy link
Contributor

kaldari commented May 25, 2014

I would be happy to fix this and submit a pull request if someone could point me in the right direction.

@pnorman pnorman added this to the New features milestone May 25, 2014
@matthijsmelissen
Copy link
Collaborator

@kaldari Which operating system do you use?

@kaldari
Copy link
Contributor

kaldari commented May 25, 2014

MacOS

@matthijsmelissen
Copy link
Collaborator

Did anyone successfully set Tilemill up in MacOS? I'm running Ubuntu myself, and most tutorials I have seen are also written for Ubuntu.

@PauloCarvalhoRJ
Copy link
Author

@math1985 MacOS is a Posix system. If the tutorials are composed by shell commands, then they should work on MacOS too given all software dependencies are installed in your system.

@matthijsmelissen
Copy link
Collaborator

The dependencies are the problem, it relies on apt-get and the Ubuntu package repositories.

@pnorman
Copy link
Collaborator

pnorman commented May 26, 2014

Did anyone successfully set Tilemill up in MacOS? I'm running Ubuntu myself, and most tutorials I have seen are also written for Ubuntu.

https://gist.github.com/springmeyer/2164897 might help.

@kaldari
Copy link
Contributor

kaldari commented Jun 12, 2014

Apparently this bug was first raised 4 years ago (https://trac.openstreetmap.org/ticket/3101)! I guess I shouldn't hold my breathe then.

@matthijsmelissen
Copy link
Collaborator

Development has been on hold for some years, because the old stylesheets were too hard to maintain. @gravitystorm has converted the stylesheets to cartocss during the last 1,5 year, and now they are a lot easier to modify. A lot of things have been solved recently (only on Github 359 resolved issues in the last 1,5 year). However, we still have a huge backlog, and we can always use more volunteers to handle the issues that are left. On trac, we are down from 400 to 159 issues (which should be 0 soon), and on Github, we also still have 267 issues left to solve.

@kaldari
Copy link
Contributor

kaldari commented Jun 13, 2014

I made a pull request, complete with a new water park icon. I'm still not going to hold my breathe though ;)

@PauloCarvalhoRJ
Copy link
Author

Thanks, kaldari, I'm eager to see the results!

All the best,
Paulo

@nighto
Copy link
Contributor

nighto commented Jun 18, 2014

Thanks @kaldari!

Just as a curiosity note (no pressure!), but because I'm also interested in this matter: how much time (aproximately) would it take to these changes appear on the main website? Days, weeks?

Thanks everyone for the hard work in this project.

@dieterdreist
Copy link

2014-06-18 16:43 GMT+02:00 Arlindo Pereira notifications@github.com:

Just as a curiosity note (no pressure!), but because I'm also interested
in this matter: how much time (aproximately) would it take to these changes
appear on the main website? Days, weeks?

when the style sheet modification goes live you can also force manually the
rerender of a given tile (add /dirty to the url of the tile and the whole
metatile (8x8) will get rerendered).

@nighto
Copy link
Contributor

nighto commented Jun 18, 2014

@dieterdreist Of course. But my question was about when should I expect that the style sheet modification goes live.

@matthijsmelissen
Copy link
Collaborator

The process is that @kaldari needs to correct his pull request (no idea how long he will take), @gravitystorm needs to approve it and roll out the next version (normally every couple of weeks, but might get delayed if he considers it a feature request), Tom Hughes needs to upgrade the carto version on the rendering servers (usually within one or two days), and the rendering servers need to re-render (couple of hours).

@Klumbumbus
Copy link

I thought he did correct the pull request with #639.

@matthijsmelissen
Copy link
Collaborator

Sorry I missed that, you're right.

@nighto
Copy link
Contributor

nighto commented Jun 20, 2014

That's what I wanted to know, @math1985. Thanks for clarifiying the process!

Once again, thanks to @kaldari, @gravitystorm and all involved in this hard work.

kaldari added a commit to kaldari/openstreetmap-carto that referenced this issue Jul 12, 2014
This change is to fix issue gravitystorm#370. It adds the necessary styles, icon,
and SQL queries to properly render water parks. It also fixes the
rendering of playgrounds in the process. (Since playgrounds have an
icon, they also need a text offset.)
@kaldari
Copy link
Contributor

kaldari commented Sep 7, 2014

Well the patch was finished months ago, but I can't seem to get any response from the project maintainer (gravitystorm). I tried pinging him on the pull request and even sent him an email, but no response so far.

@pnorman
Copy link
Collaborator

pnorman commented Sep 8, 2014

As noted in the roadmap, new features are not currently a priority - having a patch doesn't eliminate the review required when merging a PR.

@kaldari
Copy link
Contributor

kaldari commented Sep 8, 2014

pnorman: Is there anything else I can do to help the process along or should I just wait now?

@matthijsmelissen matthijsmelissen changed the title Objects with leisure=water_park not being rendered Add rendering for leisure=water_park Sep 24, 2014
@matthijsmelissen
Copy link
Collaborator

Fixed by #725.

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

No branches or pull requests