New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve palettes docs and docs automation #5619

Merged
merged 2 commits into from Dec 27, 2016

Conversation

Projects
None yet
2 participants
@bryevdv
Member

bryevdv commented Dec 27, 2016

  • re-purpose bokeh-palette to display single palettes inline
  • add bokeh-palette group to display groups of palette matrices
  • add :noindex: option to bokeh-enum
  • add complete docstrings to bokeh.palettes
  • streamline property creation in bokeh.palettes

Regarding the last, it adds a little magic, in this instance I'd very much like the palettes to be expressed as near as possible to "just the data" without alot of dressing around them.

Improve palettes docs an docs automation
* re-purpose bokeh-palette to display single palettes inline
* add bokeh-palette group to display groups of palette matrices
* add :noindex: option to bokeh-enum
* add complete docstrings to bokeh.palettes
* streamline property creation in bokeh.palettes
@bryevdv

This comment has been minimized.

Member

bryevdv commented Dec 27, 2016

adding MIGRATION for change in bokeh-palette meaning

@bryevdv

This comment has been minimized.

@bryevdv

This comment has been minimized.

Member

bryevdv commented Dec 27, 2016

ping @mattpap

@bryevdv bryevdv changed the title from Improve palettes docs an docs automation to Improve palettes docs and docs automation Dec 27, 2016

@mattpap

This comment has been minimized.

Contributor

mattpap commented Dec 27, 2016

@bryevdv

This comment has been minimized.

Member

bryevdv commented Dec 27, 2016

Thanks! I'm going to merge this in an hour or so, unless you'd like more time to review it.

@bryevdv bryevdv merged commit e74a50e into master Dec 27, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bryevdv bryevdv deleted the bryanv/palettes branch Dec 27, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment