glue-viz / glue-vispy-viewers

3-d data viewers for glue based on VisPy
http://glueviz.org/en/stable/whatsnew/experimental_3d.html#experimental-3d
BSD 2-Clause "Simplified" License
25 stars 21 forks source link

Refactor viewers to use state infrastructure #238

Closed astrofrog closed 7 years ago

astrofrog commented 7 years ago

This is going to be a large PR - it refactors the viewers to use State classes, which I'll describe shortly in the documentation. The short version is that there are now classes that hold the 'state' of a viewer and are shared by all widgets that need it, simplifying the set-up significantly. This should result in a net reduction of lines of code and very simplified logic once completed.

Things that don't work or can be optimized: