Make colour ramp have quantile bins

[Robinlovelace]

Can we please make the colour bins scale to the quartiles? Currently there are only a couple of zones in the 'greenest' bins in my updated colourscheme: http://geo8.webarch.net/pct-shiny/manchester/ Ideally we need to have quantile bins as explained here: http://www.ncgia.ucsb.edu/cctp/units/unit47/html/comp_class.html

[nikolai-b]

Made some progress on this in dbd2810 , is this what you were after?

[Robinlovelace]

Code looks good but it's not working - think my changes may have broken it. Sorry! Let me know when this up-and-running!

[nikolai-b]

Sure, I think it was when you reverted the colour ramp changes that it re-introduced some old code, easily done. I'm not totally sure I've implemented the quartiles how you wanted but now it works can you have a look and close this if it is?

[Robinlovelace]

Looks good.

[usr110]

@nikolai-b
Thanks for the changes in the legend. It looks really nice.
One thing I've noticed, however, that the y-axis doesn't correspond to the underlying values for the selected zone attribute (the range doesn't correspond to the range of the y-axis). Is this something intentionally introduced?

[Robinlovelace]

@usr110 I don't think this is intended. Please try to fix if you can...

Look forward to catching up!

Robin

[usr110]

@Robinlovelace I have fixed the errors, I believe - also the dependency with the inputs, for the legend. Could you please update the code on the server? I will then test it out.

[Robinlovelace]

I think it is updated - please test:
http://geo8.webarch.net/pct-shiny/master/
Fantastic work Ali, many thanks.

[nikolai-b]

re #15 (comment), sorry that was my mistake. I didn't realize that using isolate in render plot would mean the value never got updated (maybe because as render plot itself is only called once but it is re-evaluated constantly - the Shiny way). Closing now as this seems done.