Mantis Bug Tracker

View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000592EMStudioSerial Communicationspublic2012-06-16 00:062013-05-31 11:50
Assigned ToFred 
PlatformAllOSAllOS VersionAll
Product Version 
Target Version0.0.1Fixed in Version0.0.1 
Summary0000592: Add retries on timeouts to serial communications
DescriptionAs the summary suggests, the serial communications code should retry sending commands when they time out waiting for a response from the ECU.
TagsNo tags attached.
Issue TypeImprovement
Attached Files

- Relationships
related to 0000717resolvedDeuceEFI Multiple tables appear after ECU interrogation 

-  Notes
User avatar (0001587)
Fred (administrator)
2012-06-16 00:12

The comms code should have the option to pass in a retry count, and time out value, too. It's OK if it defaults to Xms and Ytries but you need to be able to override it from calling code.
User avatar (0001693)
malcom2073 (manager)
2012-07-21 18:26

Added retries in commit cf1429e2c27c4c76744816fddfa706f4ec4acfbf

User settable retries will come later along with more in depth settings and profiles.
User avatar (0001705)
Fred (administrator)
2012-07-22 13:37

Retries have been added, but table edits do not roll back and notify the user if retries fail. Connection doesn't drop either, which is debatable (configurable) behaviour anyway. At least it should pop up "Retried failed packet X times! Rolling back edit!" or similar. Will close when retries is integrated fully.
User avatar (0002351)
Fred (administrator)
2012-10-16 15:52

Current status, Mike? Assigning.
User avatar (0002501)
malcom2073 (manager)
2013-01-01 13:43

Fixed in a8e9a3895625d4488bf793d9a9015758e7885432

Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker