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

[Feature Request ] Add VE/AFR tables to Alpha-N strategy #1455

Closed
125jdavis opened this issue May 19, 2020 · 9 comments · Fixed by #1636
Closed

[Feature Request ] Add VE/AFR tables to Alpha-N strategy #1455

125jdavis opened this issue May 19, 2020 · 9 comments · Fixed by #1636

Comments

@125jdavis
Copy link

Alpha-N fueling algorithm doesn't have VE table or target AFR table. Currently the user tunes by changing target PW values in the TPS/RPM table. Useability would be much better if the same VE/AFR fueling tables from speed density were carried over to Alpha-N.

@rusefi
Copy link
Collaborator

rusefi commented May 19, 2020

@125jdavis how would this work?

MAP and MAF works by applying the math of target AFR and VE over measured air. How do we get measured air in Alpha-N?

@125jdavis
Copy link
Author

DSC_0004

Similar to Speed Density, but VE and Pressure get lumped together into the TPS term.

@mck1117
Copy link
Member

mck1117 commented May 19, 2020

Is this a mostly-duplicate of #1299?

@125jdavis
Copy link
Author

@mck1117 I think you would know best.

Updated equation for O2 mass = TPS * Baro Pressure * Displacement / (Gas Constant * Intake Temp)

@mck1117
Copy link
Member

mck1117 commented May 21, 2020

Instead of using TPS directly, I think you want a "VE" value looked up from a table as a function of TPS and RPM. So essentially the same as the speed density math, but with pressure removed, and VE looked up using TPS instead of MAP.

@OrchardPerformance
Copy link
Contributor

Yeah this would be a continuation of #1299
Have moved the discussion there and should we kill this ticket?

@andreika-git
Copy link
Collaborator

"VE" value looked up from a table as a function of TPS and RPM

And we already have it implemented:
image

@rusefi
Copy link
Collaborator

rusefi commented May 22, 2020

@OrchardPerformance can you please document and add TS tooltip?

@OrchardPerformance
Copy link
Contributor

I don't think this function is quite what is being meant.
This function changes the fuel tables Y axis to TPS when in speed density mode.
But only when "TPS Based" is selected as an option in the "VE auto tune" settings.

What Matt is referring to is making the standard AlphaN mode have a fuel table in VE rather than Ms pulse width.
Either way it is all covered in the other ticket.

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

Successfully merging a pull request may close this issue.

4 participants