OK...here is a patch which changes Watchdog to a spinbox and adds a
status bar item when it's enabled.  99 is disabled.  I changed the
settings name too to allow downgrading.
I didn't try to provide an "upgrade" path so first time run watchdog
will be 99 and disabled.  I could add some logic for that but not sure
we want to put "upgrade" logic in the code like that.

RRR
Mike W9MDB


On Tue, Mar 1, 2016 at 12:42 PM, Bill Somerville <g4...@classdesign.com> wrote:
> On 01/03/2016 15:23, Michael Black wrote:
>> This is updated patch to allow a user to set the watchdog timeout limit.
>>
>> I've turned the Runaway checkbox on the Configuration screen into a 
>> tri-state.
>> So, when it's partially checked (i.e. shaded) just the watchdog is
>> turned on as before (if you have watchgdog enabled now this is the
>> state that you will see).
>> Fully checked turns on the spinbox on the main window too so you can
>> adjust the limit.
>>
>> So...you can fully check, set the limit, and then partial check to
>> hide the spinbox if you want.
>> The shaded partial check should prompt people to hover over the config
>> setting and the tool tip explains it.
> Hi Mike,
>
> I haven't looked at he patch in detail but that sounds like a rather
> novel use of a tri-state check box. They are designed for tree
> hierarchies where an intermediate state is supposed to indicate children
> in both checked and unchecked state. I would say that using one for this
> sort of behaviour is non-intuitive and better modelled by three radio
> buttons in a button group.
>
> Would be simpler to just put a spin box in the settings to set the
> watchdog counter reset value. I do not favour special values and in this
> case a high value would be as good as disable rather than using zero as
> a disable option. This approach would not require an main window UI
> changes. You could add a visible counter to the status bar to show
> remaining Tx cycles before auto Tx would be disabled.
>
> 73
> Bill
> G4WJS.
>
> ------------------------------------------------------------------------------
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
> _______________________________________________
> wsjt-devel mailing list
> wsjt-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/wsjt-devel

Attachment: watchdog_patch
Description: Binary data

------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://makebettercode.com/inteldaal-eval
_______________________________________________
wsjt-devel mailing list
wsjt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wsjt-devel

Reply via email to