The Aufgeladen Ghia

[Alexander_Monday]

Mine is a twin screw and yours is a roots so the discharge temps will be higher than mine.

Post the actual log if you can (assuming MS)
Might have to zip it or link to it somewhere else.

[xzener]

Will do after work. Its a Speeduino.

[Alexander_Monday]

I'm not familiar with the speeduino to know if the logs can be read by megalogviewer.

[xzener]

They can, although I get a popup stating some graph is over limits or something like that?

[Piledriver]

If your engine is not bone stock cam/intake/heads/compression/displacement throw all the timing specs in your mental trash bin. Stock specs are for stock motors, and are only useful as a starting place, if that.

I know T4s tolerate a lot more idle timing, but they have somewhat different chamber designs.
use the idle timing feature if available, it literally varies timing to help stabilize idle speed just like an old vanagon idle stabilizer, except it works much better.

[xzener]

That is why I am baffled with this engine. I'd like to think I built this motor around the supercharger. Every aspect of this engine was chosen with recommendations of industry leaders. From the JB200 cam, to the Rasser heads with dual springs and ratio rockers. I would love to throw my understanding of VW engines out the window, but am over my head when understanding fuel injection. This is all new to me... Im used to a bone stock VW engine. From acceleration enrichment to reading logs, I am under water. It runs and drives, but feel I am doing this engine a disservice with my lack of understanding. I've learned a lot, and have come a long way, thanks to all of you. But would feel MUCH more at ease knowing someone knowledgeable tuned my engine. Hopefully I can find someone locally, but am open to live tuning. I will post up my recent logs. Hopefully you guys can help out. Cheers!

[Piledriver]

The live AFR tuning works awesome in tunerstudio, but the WBO2 delay table needs sane values, the default values, like the warm up fuel tables etc are for the developers 85 Corvette..
An aircooled engines needs are quite different, esp with port injection and heads that are hot within ~30 seconds.

I was able to develop mine quickly due to a random injection event skip bug in an early version of the ms2e sequential code.

There has been talk of a tunerstudio plugin to auto generate the table, no joy to date 15 years later.
The bug dropped the injector PW to zero momentarily, so you had a time reference in the log. (timer ran out IIRC, ms2e3.03)
I "created" a working setup on ms3 simply by putting a narrow (rpm wise) column of zero fuel in the fuel table, bracketed by the normal numbers, and just moving it in rpm a few times to get the lags at various rpm/loads. The ms code averages VE out a lot but it will give you a lean (or rich) spot that can be used for determining WBO2 sensor lag.

The autotune doesn't work quite as well as it can unless that is right. Awesome when set up correctly.

[xzener]

Here is the log and tune. Lemme know if you need anything else. First Drive with Intercooler

Mind you, at 16911, one of my oil lines (I think the return from the thermostat) blew off... Felt a little performance degradation, white smoke in the rear view. Had to pull over for a quick repair job and oil top up. Please have a look... I can use all the help I can get.

[xzener]

The live AFR tuning works awesome in tunerstudio, but the WBO2 delay table needs sane values, the default values, like the warm up fuel tables etc are for the developers 85 Corvette..
An aircooled engines needs are quite different, esp with port injection and heads that are hot within ~30 seconds.

I was able to develop mine quickly due to a random injection event skip bug in an early version of the ms2e sequential code.

There has been talk of a tunerstudio plugin to auto generate the table, no joy to date 15 years later.
The bug dropped the injector PW to zero momentarily, so you had a time reference in the log. (timer ran out IIRC, ms2e3.03)
I "created" a working setup on ms3 simply by putting a narrow (rpm wise) column of zero fuel in the fuel table, bracketed by the normal numbers, and just moving it in rpm a few times to get the lags at various rpm/loads. The ms code averages VE out a lot but it will give you a lean (or rich) spot that can be used for determining WBO2 sensor lag.

The autotune doesn't work quite as well as it can unless that is right. Awesome when set up correctly.

Is there a way mine can be corrected? Couldn't hurt right? I don't have ms3, but can probably be achieved on Speeduino with Tuner Studio.

[Piledriver]

The way I described for ms3 would work fine for ms2 or probably speeduino, assuming TunerStudio is registered.

[xzener]

Just purchased a Pro Street Flyer transmission from Benco. Very excited to install it... Also purchased a serpentine belt kit from CB Performance. Sending the crank pulley off to TheDubShop for trigger wheel. It'll be nice to have one belt to run everything. Hoping I don't have the same issues with trigger loss like I've previously fought with.

[Alexander_Monday]

I couldn't open the msq, it wants the controller definition file for speeduino 202305.
Will try more when I have time to hunt down the file.

On the log, the thing that popped out immediately to me was the baro pressure.
It is 212kpa to 214kpa across the whole log. EDIT: baro correction in % not in kpa
Not sure what is going on there?
In your general settings, what do you have selected for barometric correction?
It will affect your fueling, here are the equations as far as I know:
PW = REQ_FUEL * VE * MAP * E + accel + Injector_open_time
E = gamma_Enrich = (Warmup/100) * (O2_Closed Loop/100) * (AirCorr/100) * (BaroCorr/100)

Also, I do not see a change in accel enrichment?

As far as MAT goes, are you sure the water pump and or radiator fan are working?
Your MAT looks like when my fan gave up the ghost.

If you are using PWM idle I need to see your closed loop idle settings.

[xzener]

I have entered the realm of "I have no clue what to do next".

Yes, acceleration enrichment is not configured by me, set by Joeblow with the default tune he sends with the ECU. Customer service was half way around the world, never had real time communication to fix problems. I think I need to pay someone to help me, as there is more wrong than right it seems.

The baro pressure seems ok on my end... Not sure why you're seeing it so high. The car idles at -10psi on my boost gauge. The idle is not closed loop. I have only recently got my IAC working... but that is it. The butterfly is nearly completely closed, and the IAC really only used to allow more air to start the engine. After that it only controls the idle air really. If the engine is revved, the RPM does dip below idle RPM, but the engine usually rights itself.

The intercooler system uses air across the heat exchanger, no fan. The water does heat up eventually, but not nearly as hot as without the intercooler. I plan on replacing the tank with a bigger one.

[Alexander_Monday]

Baro correction is either none, initial reading, or second sensor.
MAP is not influenced by Baro, just the fueling equation so your MAP will read the same no matter what Baro is.
If set to none, I would assume the Baro should be 100 constant.
I would have to delve into the code to be 100% confident of that, but that is how MS does it.
If set to initial reading, the ecu reads your MAP at power on and uses that constant.
I have a second sensor, so it reads Baro in real time.
Also, correction on my part.
I should have not said it was in kpa, it is actually a percentage and inverse to kpa.
So a Baro correction reading of 212% would be 47kpa indicating a reading of -7.7# on a boost gauge.
Assuming you have baro correction set to initial reading I can only guess that at start up somehow the ecu is taking what it thinks is the barometric pressure from your MAP, but it is delayed and in fact under a vacuum at that point in code execution.
Being unfamiliar with speeduino code I hesitate to guess any further than that.

[Alexander_Monday]

The speeduino code does have a sanity check for the initial Baro reading from MAP between 108kpa and 65kpa so it should ignore the readings in the log and set Baro Correction to 100%
Still not sure why it wasn't read correctly though.

Code: Select all

// ========================================== Baro ==========================================

/* 
* The highest sea-level pressure on Earth occurs in Siberia, where the Siberian High often attains a sea-level pressure above 105 kPa;
* with record highs close to 108.5 kPa.
* The lowest possible baro reading is based on an altitude of 3500m above sea level.
*/
static inline bool isValidBaro(uint8_t baro) 
{
  static constexpr uint16_t BARO_MIN = 65U;
  static constexpr uint16_t BARO_MAX = 108U;

  return (baro >= BARO_MIN) && (baro <= BARO_MAX);
}

static inline void setBaroFromSensorReading(uint16_t sensorReading) 
{
  currentStatus.baroADC = sensorReading;
  int16_t tempValue = fastMap10Bit(currentStatus.baroADC, configPage2.baroMin, configPage2.baroMax);
  currentStatus.baro = (uint8_t)max((int16_t)0, tempValue);
}

// Should only be called when the engine isn't running.
static inline void setBaroFromMAP(void) 
{
  uint16_t tempReading = mapADCToMAP(readMAPSensor(pinMAP), configPage2.mapMin, configPage2.mapMax);
  if (isValidBaro(tempReading)) //Safety check to ensure the baro reading is within the physical limits
  {
    currentStatus.baro = tempReading;
    storeLastBaro(currentStatus.baro);
  }
}

void readBaro(void)
{
  if ( configPage6.useExtBaro != 0U  ) 
  {
    // readings
    setBaroFromSensorReading(LOW_PASS_FILTER(readMAPSensor(pinBaro), configPage4.ADCFILTER_BARO, currentStatus.baroADC)); //Very weak filter
  // If no dedicated baro sensor is available, attempt to get a reading from the MAP sensor. This can only be done if the engine is not running. 
  } else if ((currentStatus.RPM == 0U) && !engineIsRunning(micros()-MICROS_PER_SEC)) {
    setBaroFromMAP();
  } else {
    // Do nothing - baro remains at last read value & MISRA checker is kept happy.
  }
}

void initialiseMAPBaro(void) 
{
  //Initialise MAP values to all 0's
  (void)memset(&mapAlgorithmState, 0, sizeof(mapAlgorithmState));
  
  //Initialise baro
  if ( configPage6.useExtBaro != 0U  )
  {
    // Use raw unfiltered value initially
    setBaroFromSensorReading(readMAPSensor(pinBaro));
  }
  else
  {
    //Attempt to use the last known good baro reading from EEPROM as a starting point
    uint8_t lastBaro = readLastBaro();
    // Make sure it's not invalid (Possible on first run etc)
    currentStatus.baro = isValidBaro(lastBaro) ? lastBaro : 100U;
    // We assume external callers already made sure the engine isn't running
    setBaroFromMAP();
  }
}