Henry Wright

Results 43 comments of Henry Wright

Hey Great work, I had a look and have a couple of comments and suggestions: 1. I think that the AC delay on function should happen from any of the...

Hey, Just updated this to provide a much more stable VSS reading when using real sensor wheels (possibly homemade) with teeth that aren't perfectly aligned. This code update makes VSS...

> I'm struggling to see the benefit here of knowing the number of pulses in a revolution. The existing code already samples over a certain number of teeth (4 by...

As discussed, reverted VSS wheel rev decoding for now. This PR is now just the init fixes to allow re-use of unused pins.

I have no more updates to this PR. If approved please pull it in.

> @ric355 when used with include afr target, it can work like power enrichment, a feature found on other ECUs Incorporate AFR already works like this abit using the same...

I am referring to the definition in speeduino.ino Where "Include AFR" uses the O2 sensor value. ` if ( (configPage2.includeAFR == true) && (configPage6.egoType == 2) && (currentStatus.runSecs > configPage6.egoStartdelay)...

To reduce EEPROM usage by 10bytes. I removed the tables and used 5 constants. The new constants scale the agressiveness of adjustment between 0% (no adjustment) and 100% (reaching the...

Hey All, I updated this to be aligned with the master after #756 was added. But also pointing out that we are out of memory in config page 9. With...

Hi Tjeerdie, Thanks for the feedback. You are mostly correct in your assumptions. First on the Large PR. I originally had this as a smaller change to just single bank...