-
-
Notifications
You must be signed in to change notification settings - Fork 404
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
Compatibility with bokeh layout refactor #3450
Conversation
1a98ed6
to
aefd187
Compare
The main thing to settle here is how we should handle the switch from widths/heights which include axes/colorbars/legends to frame widths which apply only to the plot itself.
Edit: Consensus here was to use option 2 and add separate |
I'm going to make an attempt at explaining the width/height/frame_height/frame_width
These options take absolute precedence over everything, if a fixed height or width is specified by a user no other option including AspectThe Data aspectThe ResponsiveIn
There is one major caveat when combining a data aspect and responsive mode, in this case we cannot fully guarantee that a specified data aspect is maintained. This is because in responsive mode we don't have control over the plot frame dimensions just the overall plot aspect. We may be able to alleviate this by writing some JS code that maintains equal aspect in the plot ranges or by dynamically syncing the plot frame extents. Updating PlotsThe other major issue here is maintaining a fixed aspect while a plot is updating. We do not want a plot resizing when you update it (in fact I could not even get that working), so the only way to achieve a fixed |
72a9e68
to
b8db63b
Compare
aac3a41
to
911ed98
Compare
I know we discussed all of the above comments at some point, but my brain starts to hurt when trying to really think deeply about them now. That said, as far as I can see that all sounds like what we had discussed and seems like the best approach in each case. |
I agree with Jim that this sounds just like we discussed. From the description, it sounds like everything behaves the way I would hope, except maybe for the caveat about |
Feature request: plot_size option for GridSpace should support tuples |
db498b8
to
1821c1e
Compare
254ccaa
to
385af52
Compare
625b9b1
to
8bc9340
Compare
@philippjfr Looks great! Only had one minor request (docstring improvement) and other than that, I just want a way to remember to remove Happy to merge once the small change is pushed (no need to wait on the tests this time). |
Great thanks! Merging. |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This PR achieves compatibility with the layout refactor which was merged into bokeh dev and will land in bokeh 1.1.0. It will give us control over the actual plot frame dimensions (excluding the axes, colorbars, legends) and provide control over the plot aspect.
aspect
anddata_aspect
in bokeh (Separating options to set figure vs. data aspect ratio #335)Fixes #3026
Fixes #2992