BioSimSpace icon indicating copy to clipboard operation
BioSimSpace copied to clipboard

Set ipywidgets<8 to Avoid Issues with NGLView

Open fjclark opened this issue 3 years ago • 4 comments

Hello,

Recent changes to ipywidgets have broken NGLView - ensuring version <8 avoids any issues.

Thanks, Finlay

fjclark avatar Sep 01 '22 15:09 fjclark

Thanks for this. Are the NGLView developers aware of this? If not, can you post and issue there too. While this workaround is good for now, the ideal solution is for NGLView to pin in their own conda recipe and release a new version built against ipywidgets 8.

Do you know if this also breaks any of our own functionality based on ipywidgets, e.g. interactive nodes with gui elements?

Cheers.

lohedges avatar Sep 01 '22 16:09 lohedges

Sorry, missed your link to the bug report. I think we can use this workaround for now and update if/when NGLView fix the issue upstream.

Cheers

lohedges avatar Sep 01 '22 16:09 lohedges

Great. There don't seem to be any issues with interactive nodes with GUI elements - just with NGLView.

Cheers

fjclark avatar Sep 01 '22 16:09 fjclark

Thanks for confirming. We only use pretty basic widgets so I'm glad it's unaffected. Should hopefully be able to merge this in tomorrow.

lohedges avatar Sep 01 '22 16:09 lohedges