Mantis Bug Tracker

View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000251TunixFreeEMS Pluginpublic2011-08-27 18:372011-11-18 20:56
ReporterFred 
Assigned Todandruczyk 
PrioritynormalSeverityminorReproducibilityalways
StatusassignedResolutionopen 
PlatformAllOSAllOS VersionAll
Product Version 
Target Version0.9.24Fixed in Version 
Summary0000251: App Swallows FreeEMS Errors Instead Of Displaying Them To The User
DescriptionI don't get popups when i intentionally cause a firmware error response, instead of the desired action, nothing happens. In general, if there is an error from the firmware, i want the user to see it.

In the case where its repetitive, ok, there is a problem that needs solving, and maybe a "don't show this error again in the current session" option would be good. But they really need to be told what is going on.

In most cases an error from the firmware = incompatible, a bug in mine or a bug in yours, all of which we need to know about ASAP, and having the user say "got error 345" is a good way of finding out AND understanding why in one go.

In other cases, though, it's legitimate that you get the error, and it tells you that you did something wrong using the app/fw, a wrong value or some such thing. Those errors are user errors as much as they are dev errors and should be passed on.

We should have a set of pretty messages for them, and display number, name, and message in a dialog and log it to a file also. We need to hear about it, and that's the only way it'll happen, IMO.

Right now you output the error to the console, that'll do for now, but the user, who may launch from a menu, should see it too.
Steps To ReproduceEnter value of 2-100 in any PW field of the bench test decoder tab and hit send.
TagsNo tags attached.
FirmwareVersion
Attached Files

- Relationships

-  Notes
(0000239)
dandruczyk (viewer)
2011-08-27 22:08

Dude, look in hte terminal where mtx is started


Packet ERROR Code 0x7778, "Too short of a PulseWidth to test!"
ERROR with Bench test packet!

So its not in the Gui YET.. it DOES WORK.
User avatar (0000240)
Fred (administrator)
2011-08-28 14:23

From the issue body: "Right now you output the error to the console, that'll do for now, but the user, who may launch from a menu, should see it too."

Yep, I noticed that! :-) And yep, great that it's there! This is just a TODO item, really.
(0000243)
dandruczyk (viewer)
2011-08-28 15:15

added messagebox to the benchtest window that reports issues to the user specific to those packets.
User avatar (0000244)
Fred (administrator)
2011-08-28 15:34

Excellent, tested and works well! :-)

Do you plan to have errors broken down into different categories with different handlers like an exception hierarchy? and only allow "unknown" ones to float up to a popup? if so, that would be very cool. ie, bench test stuff goes to the little text area, and other things to various other places such as warning lights in some fancy gui UI and random stuff that shouldnt happen comes to a popup?
User avatar (0000613)
Fred (administrator)
2011-11-18 19:57

Bump, I'd like to see this being more obvious/annoying with a window to display certain bad ones and a list like the bench test one somewhere where we can watch it all the time, a small separate window maybe? One vote for more work on this! :-)
(0000616)
dandruczyk (viewer)
2011-11-18 20:44

You bitched left and right about the popup errors, so I'm not going back to that....
User avatar (0000617)
Fred (administrator)
2011-11-18 20:56

I bitched about pop up errors that can't be turned off on an individual basis with hard coded behaviour, Dave! Especially about ones that I couldn't give two hoots about, and that are part of features that I never use...

Besides, if you read the comment you'll see that i suggested other things. Anything should be suppressible...

Don't assign back to me or change the status until this is handled in some form :-p


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker