type = "tile" (alias "heatmap" - #677
Conversation
|
Some very quick feedback only based on the examples above. (I didn't have time to look at the code or try the examples myself, yet.)
|
|
Thanks for the quick feedback @zeileis.
You mean you'd like to drop "heatmap" even as an alias?
👍
Yes, we don't do any checks or transformations. (I think this is probably the right approach.)
Agree, but I haven't really thought how we could do that. Unless I'm missing something, we'd need some custom logic on top of
Hmmm. The user can just pass |
Actually, an easy solution is just to identify the midpoint and then expand the limits symmetrically around that midpoint. BUT the trick is deciding how users should be able to pass through this midpoint to the palette ( On reflection, I think that this issue should be tackled in a separate PR as part of additional palette customization entry point. Maybe a sister, top-level |
|
I've been thinking about this some more and I agree that this should probably go into a separate issue/PR. Is there currently the possibility to control the range of the This would be another option to get the midpoint exactly at 0, namely by setting the This would be useful more generally, e.g., when I want to use exactly the same |
|
Regarding the name: Yes, I would drop the |
Closes #665
Some quick examples from the documentation:
# restore theme tinytheme()