redwagon wrote:
Ecuflash
Fueling - Warm-Up Enrichment group contains no tables.
None have been defined yet.
redwagon wrote:
Romraider
Fueling - Warm-Up Enrichment group is absent altogether
None have been defined yet.
redwagon wrote:
Fueling - Primary Open Loop Fueling Cruise is scaled oddly... rpm axis goes to 4000rpm in 7 rows then the last 4 are zeros. Could be it's supposed to be this way with extra rows but these extra rows are not all one value as is usually the case when Subaru wastes space in tables. Ecuflash definition looks much more normal with only 7 rows.
Fueling - Primary Open Loop Fueling Non-Cruise is scaled oddly. I would guess the table should only have 12 columns like the Ecuflash def.
The X & Y axis sizes in the RR def had been interchanged, fixed in v2 def
redwagon wrote:
Timing Compensation (IAT) Activation table has two identical values in the engine load scale but the values in these two rows are different. Seems a little odd.
Yes that's odd and it is how the values are set in the ROM. It is still used in the logic, but now acts more like an on/off switch for "Timing Compensation IAT" more than anything.
redwagon wrote:
Intake Cam Advance Angle Non-Cruise I can enter a minimum value of -40* which is possible I guess, but maximum value can be set up to 215*, which cannot be mechanically possible.
Exhaust Cam Retard Angle Non-Cruise I can enter a minimum value of 0* which is possible I guess, but maximum value can be set up to 255*, which cannot be mechanically possible.
This is the same behaviour as any other ROM and any other table. The min/max is enforced only by the acceptable values for the particular data type of the field. i.e. unit8 range is 0 to 255, uint16 range is 0 to 65535. This range is then used in the table data formula to determine the limits. So for the intake AVCS with the data type uint8 and expression="x-40" the min is "
0-40" and the max is "
255-40". I guess as a tuner you already know the 215 degrees of advance is not possible. Hence tuning is not for the uneducated
Original post has been update with a new RR def. The EcuFlash def was correct all along.