LemurPwned / VISM

Software for visualising magnetic layers
Other
6 stars 2 forks source link

Better2DGraph widget issue #29

Closed pawelkulig1 closed 6 years ago

pawelkulig1 commented 6 years ago
LemurPwned commented 6 years ago

I will try to work this issue out while pull requests are not fully resolved

LemurPwned commented 6 years ago

I think this issue has fully resolved, if not that is not a problem solely of that particular 2d widget but general window resizing

pawelkulig1 commented 6 years ago

Fixing windows resizing problem haven't solved that. I will try to find solution once again.

pawelkulig1 commented 6 years ago

I think I have found solution to fix this, this may take me a lot of time to resolve, for now I would recommend to exclude this widget, from available widgets because we have another working widget that is working properly.

LemurPwned commented 6 years ago

Solution: the pyqtgraph widget has been created twice -> once because the Canvas2d upgraded class has inherited PlotWidet and then called the default constructor of a parent class, and secondly, this time by intention, in the init function of child class. This caused the two to overlay and thus cause problems. Removing inheritance on pyqtgraphs and directly inheriting from AbstractCanvas class allowed for easy handling of the size (probably can cause some conflict later with matplotlib context being invoked as well).