Mantis Bug Tracker

View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000580EMStudioGeneralpublic2012-06-10 22:212012-07-21 23:54
ReporterFred 
Assigned Tomalcom2073 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformAllOSAllOS VersionAll
Product Version 
Target Version0.0.1Fixed in Version0.0.1 
Summary0000580: locations with ecu ram should change on edit and burn with save
Descriptionsave = write device ram to flash internally, NOT send full block with flash write cmd.

edits should take immediate effect
TagsNo tags attached.
Issue TypeNew Feature
Attached Files

- Relationships

-  Notes
User avatar (0001544)
malcom2073 (manager)
2012-06-10 22:27

This is a functionality feature, so it is not super high priority, but it will eventually be an option.
User avatar (0001546)
Fred (administrator)
2012-06-10 22:29

Disagree, this is a basic and fundamental behaviour. When you fix it, up to you, but it's very important that it works that way. First step, grey/hide/rm save button and make it live edit. Second step, connect save to burn packet.
User avatar (0001548)
malcom2073 (manager)
2012-06-10 22:33

Agree to disagree. I will add it as the default option, but maintain the option to have it the way it currently is.
User avatar (0001549)
Fred (administrator)
2012-06-10 22:43

It's valid to edit without sending, but only when "offline" at all other times you definitely want to send to ram. Just as you definitely don't want to write directly to flash ever. Possible exception for bench test ram edit. This is a fixed desired behaviour that is common to all users, not something that user A wants one way and user B wants another. Trust me. I know this shit. :-)
User avatar (0001551)
malcom2073 (manager)
2012-06-11 08:22

I trust you, however I also trust my own experience with tuning where I know how I personally have wanted it. I'll make your method the default until someone complains, but it will still be a setting :)
User avatar (0001552)
Fred (administrator)
2012-06-11 08:24

Tell me why you wanted it?
User avatar (0001553)
Fred (administrator)
2012-06-11 08:25

No one will ever complain, though...
User avatar (0001556)
Dan (reporter)
2012-06-11 11:49

I must say i agree with fred on this one!
User avatar (0001557)
malcom2073 (manager)
2012-06-11 12:31

When I first started tuning my engine, I had to adjust the entire stock timing table at once to give it a much more gradual curve. Granted I did it with the engine off (Though I was still connected to my ecu), but had I done it with the engine on, I would have liked to get the table adjusted to a nice starting point, and then write it all at once. Especially if you typo inside a value, and suddenly your engine starts running like crap. I've done that once or twice.

I see a use for it, so I don't see a reason to get rid of it as a feature.
User avatar (0001558)
Dan (reporter)
2012-06-11 12:44

But isnt it better to adjust a single value at the point where the engine is currently running to see whether you improve or begin to screw it up before you apply all changes at once. If you apply changes all at once, then you could seriously damage the engine being tuned.
User avatar (0001608)
Fred (administrator)
2012-06-24 02:55

Fixed ages ago. Tested tonight in 257f8e043810ec376296b9cc09d0be5ea47fe5f1


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker