Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RPM correction/multiplier for Accel Enrich #4760

Closed
ElDominio opened this issue Nov 8, 2022 · 9 comments
Closed

RPM correction/multiplier for Accel Enrich #4760

ElDominio opened this issue Nov 8, 2022 · 9 comments
Assignees

Comments

@ElDominio
Copy link
Collaborator

add a 2D RPM correction table to transient enrich, to be able to lower the amount of enrich applied at upper RPMs (since engines in motion usually need much less enrichment).

A table like this
image

maybe even with 4 cells is enough. 1.0 at 1000RPM, .9 at 3000rpm, .5 at 5000RPM, 0 at 7000RPM.

@mck1117
Copy link
Member

mck1117 commented Nov 8, 2022

how much time do you spend at 7k rpm while below 30c coolant?

@ElDominio
Copy link
Collaborator Author

ElDominio commented Nov 8, 2022

no coolant just showed that to show what a 2D table looks like. Coolant would be replaced by RPM

4 cells, Coolant is now RPM.

Left side is RPM, right side is Multiplier

@mck1117
Copy link
Member

mck1117 commented Nov 8, 2022

yes I understand the request - I'm asking why you think it's necessary

@ElDominio
Copy link
Collaborator Author

ElDominio commented Nov 8, 2022

adding 3ms on a transient at 5.500rpm usually just makes an engine lug

you could be decelerating coming off a corner, mash the gas at 5.500 and if the TPS rate matches the same you'd see at idle, you'll get the same amount of fuel added

@mck1117
Copy link
Member

mck1117 commented Nov 8, 2022

oh duh the snip of a random table confused me

@ElDominio
Copy link
Collaborator Author

image

@mck1117 mck1117 self-assigned this Nov 8, 2022
@ElDominio
Copy link
Collaborator Author

Also, a CLT-based correction on enrich (more enrich added when colder) would also be good. Some ecus have a "Cold/Hot" temp, where the Hot temp is the normal, and the cold temp uses a multiplier.

Example:
Cold = 100F ; Multiplier = 1.3
Hot = 150F

(up to 100F, it uses 1.3 as a multiplier, then interpolates between 100F and 150F to become 1x multiplier.)

@ElDominio
Copy link
Collaborator Author

@mck1117 @rusefillc any progress?

@rusefillc
Copy link
Contributor

Some progress

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants