Open karthik opened 10 years ago
Saw this after I sent my pull request.
Hmmm, may not make sense to add mine in given you have plans for new palettes...
Thanks Jeff! Ah, no worries. I'll review this PR next week and get back to you.
Sounds good.
On Fri, Jul 25, 2014 at 12:31 PM, Karthik Ram notifications@github.com wrote:
Thanks Jeff! Ah, no worries. I'll review this PR next week and get back to you.
— Reply to this email directly or view it on GitHub https://github.com/karthik/wesanderson/issues/3#issuecomment-50172997.
Jeff W. Hollister email: jeff.w.hollister@gmail.com google voice: 401 326 2531 cell: 401 556 4087
https://github.com/hadley/wesanderson/tree/palette-list implements the first two - happy to turn it into a pull request
@hadley I looked this over and it would be great to have as a PR. Please send along when you find some time. Thanks!
Based on discussions with Hadley, I'm going to rework the package to stop mimicking the same style as
RColorBrewer
.S3
methods to justplot
any paletteget
If anyone has other suggestions, please add here. I'll leave this version as-is for
RColorBrewer
fans and bump up to version 2.