Summer work on the planner - Issue with QSpinBox planner preferences

[Stefan Fuchs]

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 <tcanabrava at kde.org
>>> <mailto:tcanabrava at kde.org>> 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: sfuchs at gmx.de <mailto:sfuchs at gmx.de>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20170918/079b9dab/attachment.html>