CA18DET Variable Intake Solenoid scale bugs

[louiswun]

As title,
RPM scale on the table is not the same as map.
Load scale seems something wrong.

[Matt]

http://www.nistune.com/mantis/view.php?id=59
Now fixed for next release

[MechanisSan]

It seems to be still something wrong with that. Now the X-axis is called "load" but it seems to be rpm, but with factor 2 too high.
Currently with untouched oem values the soleniod does switch already above approx 1800rpm when there is also load. But that does not make any sense. The system is designed to switch above approx 3700 rpm when under load.
I tested a bit with the values, and like I mentioned it seems to be just a wrong scale (factor 2) at the rpm's.

[MechanisSan]

That's how it still looks like:
Look at the selected row. In the table it's at 3200 rpm, but in the graph it's at 1651 rpm. The numbers from the graph are where it's actually switching.

butterfly_control.jpg

(298.32 KiB) Downloaded 2011 times

[Matt]

I'll fix that in the next update. I moved all the definitions from hardcoded to XML file, and checked through most tables but some might have got missed

[MechanisSan]

Ok great. I've found some other small faults where "filtered" and "raw" is confused. I'll post the details of it later.

[MechanisSan]

At
"Idle Timing Offset warm (Temp)"
and
"Timing Beta Advance vs Temp"
the RAW / filtered states are confused

[Matt]

'Filtered' isn't displaying on the grid. I'll look into that

[Matt]

Those two tables shouldn't have 'filters' applied

CA18DET is one of those strange ECUs which has an offset of 16 applied to some of the idle timing tables. It doesn't show in other ECUs, because they dont have offsets

Anyway it shouldn't apply filtering to these offset tables, so I've added an extra category in the XML file to deal with IDLE_TIMING_OFFSET tables like these. Fixed along with the variable intake table for next release which should hopefully be out next week

[MechanisSan]

Thanks Matt, appreciate the support