wesanderson

wesanderson copied to clipboard

Based on discussions with Hadley, I'm going to rework the package to stop mimicking the same style as RColorBrewer.

• [ ] I'll just add S3 methods to just plot any palette
• [ ] Avoid using get
• [ ] Add a whole slew of new palettes I've been painstakingly assembling (no longer from the tumblr)

If anyone has other suggestions, please add here. I'll leave this version as-is for RColorBrewer fans and bump up to version 2.

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 [email protected] 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: [email protected] 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!