Hi Robert, hi Tomaz,
Am 09.09.2017 um 22:11 schrieb Stefan Fuchs: > Am 30.08.2017 um 19:45 schrieb Robert Helling: >> >>> On 30. Aug 2017, at 10:18, Tomaz Canabrava <[email protected] >>> <mailto:[email protected]>> wrote: >>> >>> Robert, >>> >>> What is still missing: >>> A ui element to turn this expensive calculation off. >>> >>> >>> This is easy to add, wouldn't take me much time. >> > I have a finding here which may be related to this discussion about > long calculation times for Roberts new feature and the questions about > plan() being called multiple times: > > Since Roberts change here on my side not only the calculation of the > plan takes much longer but also this happens: > If I increase or decrease any QSpinBox value in the planner > preferences by clicking on the up or down arrow the QSpinBox value is > increased/decreased by two steps instead of one. This happens with any > QSpinBox based value like gas options, ascent rates, VPM conservatism,... > > I don't believe this really comes from Roberts change but it was maybe > revealed by it. > > Can you reproduce this? > I found a hint how this double increment/decrement issue of the QSpinBoxes could be explained: https://github.com/MRtrix3/mrtrix3/issues/306 So beside the fact that I up to now do not like these long calculation times after each parameter change that much (I guess it's the same for Robert himself) I would recommend the following regarding the UI controls: - Turn off "keyboardTracking" for any affected UI element in the planner to avoid calculations being triggered WHILE the user types in a value. This is easy change. - Turn off the mouse button hold causes repeated in/decrease feature of each QSpinBox. No clue yet how this can be done. Best regards Stefan -- Stefan Fuchs E-Mail: [email protected] <mailto:[email protected]>
_______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
