RomRaider Logo

RomRaider

Open Source ECU Tools
 FAQ •  Register •  Login 

RomRaider

Documentation

Community

Developers

It is currently Tue Dec 23, 2025 10:59 am

All times are UTC - 5 hours [ DST ]





Post new topic Reply to topic  [ 140 posts ]  Go to page Previous  1 ... 6, 7, 8, 9, 10
Author Message
 Post subject: Re: RomRaider Learning Table Values feature
PostPosted: Sun Jun 29, 2025 10:06 pm 
Offline
Newbie

Joined: Tue Nov 21, 2023 11:06 pm
Posts: 16
dschultz wrote:
The ECU def specifies the wrong number of rows for your FLKC table.
It should be 4 rows and 5 columns for a total of 20 cells in the table.
You can change your ECU def, find "Fine Correction Rows (RPM)" including the quotes. The first defines only the address for your ECU, the second occurrence is the table definition.
There are two entries in that section for sizey="6", change both to 3
Then delete the three lines:
Code:
       <data> Max Range 4 / Min Range 5 </data>
        <data> Max Range 5 / Min Range 6 </data>
        <data> Max Range 6 / Min Range 7 </data>
It should look like this after.
Code:
    <table type="2D" name="Fine Correction Rows (RPM)" category="Ignition Timing - Knock Control" storagetype="float" endian="little" sizey="3" userlevel="4">
      <scaling units="Engine Speed (RPM)" expression="x" to_byte="x" format="#" fineincrement="50" coarseincrement="100" />
      <table type="Static Y Axis" name="Fine Correction Stored/Applied RPM Ranges" sizey="3">
        <data> Max Range 1 / Min Range 2 </data>
        <data> Max Range 2 / Min Range 3 </data>
        <data> Max Range 3 / Min Range 4 </data>
      </table>
      <description>These are the engine speed ranges that make up the fine learning correction table stored in RAM. These rpm ranges, as well as the load ranges specified by the 'Fine Correction Columns (Load)' table, determine how each fine correction value is stored as well as applied.</description>
    </table>

Save the file.

This is the raw data from the ROM I have for the rows and columns, did you change yours?
Attachment:
rc.png
Attachment:
rc1.png

If so you need to reset them and reflash the ECU with the updated values.


I was able to make the change to the ECU def. I haven't touched the ECU def prior to this, so I don't know how it ended up with extra rows. Can you post the ROM that you have? I want to compare it to what I have. I'm also confused in ECUFlash. If there are only 3 RPM ranges in the ROM, why is ECUFlash showing 6? Does the 32bitbase need to be corrected for ECUFlash?


You do not have the required permissions to view the files attached to this post.


Top
 Profile  
 
 Post subject: Re: RomRaider Learning Table Values feature
PostPosted: Tue Jul 01, 2025 3:17 pm 
Offline
RomRaider Developer

Joined: Wed May 20, 2009 9:49 pm
Posts: 7314
Location: Canada eh!
ROM is here.
Yes the EcuFlash 32bit base will need to be modified as well. There have only been 4 RPM rows (down from 7) in the FLKC table since about 2015 but no one has made a new base table and corrected the posted defs.

This image is the ECU code that confirms the number of rows/columns (0 base) populated.
Attachment:
rows_cols.png

Your car doesn't rev to 10000+ does it?


You do not have the required permissions to view the files attached to this post.


Top
 Profile  
 
 Post subject: Re: RomRaider Learning Table Values feature
PostPosted: Tue Jul 01, 2025 11:35 pm 
Offline
Newbie

Joined: Tue Nov 21, 2023 11:06 pm
Posts: 16
dschultz wrote:
Yes the EcuFlash 32bit base will need to be modified as well. There have only been 4 RPM rows (down from 7) in the FLKC table since about 2015 but no one has made a new base table and corrected the posted defs.

Your car doesn't rev to 10000+ does it?


I think I can get my car to rev to 10000+............once! I did not know the number of RPM rows was reduced after 2015. Thanks for providing that information. Almost there. I have the address for E173 set to 0xFF33B0 and I have the ECU def corrected. I'm still getting incorrect info in the last row of the LTV table. That is a B and a zero in the address, correct? My eyes are getting worse as I get older.


You do not have the required permissions to view the files attached to this post.


Top
 Profile  
 
 Post subject: Re: RomRaider Learning Table Values feature
PostPosted: Thu Jul 03, 2025 8:33 pm 
Offline
RomRaider Developer

Joined: Wed May 20, 2009 9:49 pm
Posts: 7314
Location: Canada eh!
b zero is correct.

This is the response for the last row:
21:27:49,262 DEBUG [SwingWorker-pool-2-thread-1] - Mode:STATE_0 ECU Response <--- 80F01015E8 C2FA0000 C2FA0000 C2FA0000 C2FA0000 C2FA000029

The value C2FA0000 is a IEEE745 float representation of -125.
It's possible that the last row is not used, a more in-depth review of the ECU code is required to figure out what is going on.


Top
 Profile  
 
 Post subject: Re: RomRaider Learning Table Values feature
PostPosted: Mon Jul 07, 2025 2:57 pm 
Offline
Newbie

Joined: Tue Nov 21, 2023 11:06 pm
Posts: 16
dschultz wrote:
b zero is correct.

This is the response for the last row:
21:27:49,262 DEBUG [SwingWorker-pool-2-thread-1] - Mode:STATE_0 ECU Response <--- 80F01015E8 C2FA0000 C2FA0000 C2FA0000 C2FA0000 C2FA000029

The value C2FA0000 is a IEEE745 float representation of -125.
It's possible that the last row is not used, a more in-depth review of the ECU code is required to figure out what is going on.


I built a solution of ScoobyRom to start looking around and learning more as I have time. Thanks for all the help so far. Much appreciated!


Top
 Profile  
 
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 140 posts ]  Go to page Previous  1 ... 6, 7, 8, 9, 10

All times are UTC - 5 hours [ DST ]


Who is online

Users browsing this forum: No registered users and 5 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Jump to:  
Powered by phpBB © 2000, 2002, 2005, 2007 phpBB Group
Style based on FI Subsilver by phpBBservice.nl