methods of QtInteractor

[pyvista]

methods of QtInteractor

[GuillaumeFavelier]

Hi @bistek, thanks for doing this manual testing and I'm surprised by the outcome. I think it could be a good idea to have automatic testing for this then once it is fixed.

Among the methods crashing, add_toolbars() and add_menu_bar() are not inherited from BasePlotter. Also AFAIK, default_camera_tool_bar is not a function.

[akaszynski]

Looks like we indeed need to add automated testing for this. Thanks for discovering this!

[andrea-bistacchi]

I'm happy to help, if I can. Actually YOU are the one who are doing a great job for the community!

[dominicjtriana]

I made another issue but I would add:
self.plotter.disable_eye_dome_lighting() throws errors but doesn't crash. There are some negative visual artifacts as well.

[condoratberlin]

Hi, I goning to embedd pyvistaqt in an gui framework an discovered some similar problems.
Especially add_toolbars() requires a self.app_window attribute, were the mainwindow is stored, but the pyvista QtInteractor don't allow to set such a parameter to the init method.

[GuillaumeFavelier]

I agree with @condoratberlin in here, the add_toolbars() and add_menu_bar() functions depends on app_window internally. Those should not be declared as QtInteractor methods.

I see two ways to deal with this at the moment:

1. Just move those 2 to BackgroundPlotter (that's where they are used after all)
2. Find a way to refactor the functions to make them useful to QtInteractor (requires Qt gymnastics)

About EDL, this was fixed in pyvista/pyvista#1113

[GuillaumeFavelier]

Option 1 was implemented in #78

[GuillaumeFavelier]

As of now, is there any confusion about the methods of QtInteractor?