Anonymous | Login | Signup for a new account | 2019-02-22 03:59 UTC | ![]() |
Main | My View | View Issues | Roadmap | Repositories |
View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0000580 | EMStudio | General | public | 2012-06-10 22:21 | 2012-07-21 23:54 | ||||
Reporter | Fred | ||||||||
Assigned To | malcom2073 | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | All | OS | All | OS Version | All | ||||
Product Version | |||||||||
Target Version | 0.0.1 | Fixed in Version | 0.0.1 | ||||||
Summary | 0000580: locations with ecu ram should change on edit and burn with save | ||||||||
Description | save = write device ram to flash internally, NOT send full block with flash write cmd. edits should take immediate effect | ||||||||
Tags | No tags attached. | ||||||||
Issue Type | New Feature | ||||||||
Attached Files | |||||||||
![]() |
|
![]() 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. |
![]() 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. |
![]() 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. |
![]() 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. :-) |
![]() 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 :) |
![]() Fred (administrator) 2012-06-11 08:24 |
Tell me why you wanted it? |
![]() Fred (administrator) 2012-06-11 08:25 |
No one will ever complain, though... |
![]() Dan (reporter) 2012-06-11 11:49 |
I must say i agree with fred on this one! |
![]() 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. |
![]() 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. |
![]() Fred (administrator) 2012-06-24 02:55 |
Fixed ages ago. Tested tonight in 257f8e043810ec376296b9cc09d0be5ea47fe5f1 |
Copyright © 2000 - 2011 MantisBT Group |