Skip to content

Filled in missing Documentation for Heatmap, Marks(Pie), Marks(market), scales, axes etc. Considered previous style and matched it correctly#1696

Open
ishaan-cherukuri wants to merge 1 commit intobqplot:masterfrom
ishaan-cherukuri:master

Conversation

@ishaan-cherukuri
Copy link
Copy Markdown

References

Code changes

User-facing changes

Backwards-incompatible changes

Comment thread main.py
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose this was committed by mistake?

@martinRenou
Copy link
Copy Markdown
Member

That's amazing thanks! I'll give this a review

Copy link
Copy Markdown
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a ton, apart from one comment + the unwanted commited files that's good to go

opacity_slider = widgets.FloatSlider(value=1.0, min=0.0, max=1.0, step=0.05,
description="Opacity")
# jslink works on simple scalar traits
widgets.jslink((opacity_slider, "value"), (fig, "fig_margin")) # example only
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know it's only an example, but it is likely incorrect, fig_margin is not a scalar value but a dictionary like dict(top=60, bottom=100, left=60, right=60)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants