Skip to content
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

Unexpected behavior when deleting from layout.children #5967

Closed
paddymul opened this issue Mar 8, 2017 · 4 comments · Fixed by #5976
Closed

Unexpected behavior when deleting from layout.children #5967

paddymul opened this issue Mar 8, 2017 · 4 comments · Fixed by #5976

Comments

@paddymul
Copy link
Contributor

paddymul commented Mar 8, 2017

I have encountered a bug when updating row contained in a column.

When r1.children[0] is deleted, r2 disappears and only the tools are left. I would expect a column with s1 above s2. Is deleting from the children array not supported?

Example IPython notebook https://github.com/paddymul/bokeh_bugs/blob/master/Bokeh%20row%20update%20artifact.ipynb

#this messes up the rendering row2
r1 = row(s1,s2)
r2 = row(s3)
displayed_obj = column(r1, r2)

handle = show(displayed_obj, notebook_handle=True)
time.sleep(5)
del(r1.children[0])
push_notebook(handle=handle)

screen shot 2017-03-08 at 4 40 31 pm

Versions
python 2.7.11
IPython notebook 4.1.2
Bokeh 0.12.4

Thanks

@mattpap
Copy link
Contributor

mattpap commented Mar 9, 2017

Modifications of layout after initial rendering aren't well supported right now. This issue is likely a duplicate, but I will leave it open as there is substantial feedback here.

@bryevdv
Copy link
Member

bryevdv commented Mar 24, 2017

I agree it's probably a dupe of the other issues. I've moved it to 0.12.6 just to make sure it gets tracked in the next release cycle.

@bryevdv
Copy link
Member

bryevdv commented Apr 26, 2017

@mattpap do you have a test case for this? The original linked code is a 404 now.

@bryevdv
Copy link
Member

bryevdv commented Apr 26, 2017

Confirmed seems to be working now with this code

screen shot 2017-04-26 at 14 54 49

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

Successfully merging a pull request may close this issue.

3 participants