search

CartoDB / Windshaft

A Node.js map tile library for PostGIS and torque.js, with CartoCSS styling
BSD 3-Clause "New" or "Revised" License
308 stars 82 forks source link

Allow setting colors from table data #287

Open iriberri opened 9 years ago

iriberri commented 9 years ago

It seems that this is possible from one year ago: https://github.com/mapbox/tilemill/issues/1809

Could we update the system to support it?

danicarrion commented 9 years ago

+1

andrewbt commented 9 years ago

+1 This has now been requested by a partner and would also be useful for getting around the category wizard's default upper limit of 10 categories. @saleiva @javisantana thoughts? Seems like just a simple update to mapnik source.

javisantana commented 9 years ago

"simple" and "mapnik" can't go in the same sentence man :)

rochoa commented 9 years ago

Just an small update, I looked a bit into this and as stated above is not that simple. Something in our stack makes this not to work with latest version. Using same cartocss version linked in tilemill and windshaft.

I have to dedicate more time to this one.

iriberri commented 9 years ago

Ha! :-) Thanks for the update Raúl!

saleiva commented 9 years ago

We need a better estimation for this - @rochoa ?

rochoa commented 9 years ago

Next milestone.

saleiva commented 9 years ago

I meant in effort :)

rochoa commented 9 years ago

It's about debugging, I cannot estimate debugging time.

saleiva commented 9 years ago

Then, I'm not sure I'd prioritize this so much, unless it's related with some other bugs

rochoa commented 9 years ago

If you agree, I will try to invest a fixed amount of time depending on the rest of the milestone to at least document what's going on.

peterdesmet commented 9 years ago

I'd also like this feature. In fact, I'd like to do a RGB mix based on table data:

polygon-fill: rgb(value1, value2, value3)
rochoa commented 9 years ago

Well, just to summarize the changes we need to make to support this:

Parsing

Rendering