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

wow we have two separate per-cylinder trims?! #5237

Closed
rusefillc opened this issue Apr 12, 2023 · 4 comments
Closed

wow we have two separate per-cylinder trims?! #5237

rusefillc opened this issue Apr 12, 2023 · 4 comments
Labels

Comments

@rusefillc
Copy link
Contributor

we have per-cylinder table

angle_t getCylinderIgnitionTrim(size_t cylinderNumber, int rpm, float ignitionLoad) {
	return interpolate3d(
		config->ignTrims[cylinderNumber].table,
		config->ignTrimLoadBins, ignitionLoad,
		config->ignTrimRpmBins, rpm
	);
}

and we have

	// Plus or minus any adjustment if this is an odd-fire engine
	auto adjustment = engineConfiguration->timing_offset_cylinder[cylinderNumber];
@rusefillc
Copy link
Contributor Author

@mck1117 @ElDominio is this a feature or a bug?

rusefillc pushed a commit that referenced this issue Apr 12, 2023
only renaming variable here
@mck1117
Copy link
Contributor

mck1117 commented Apr 12, 2023

Feature! The cylinder offset is for your crankshaft (odd fire engines, etc) and the per cylinder tables are for tuning.

@ElDominio
Copy link
Collaborator

I used it for 180* angle 2 cylinder, can confirm

@mck1117
Copy link
Contributor

mck1117 commented Apr 12, 2023

Mark uses it for Harley too. The timing mark is actually not lined up any cylinder, one is 22.5 ahead and the other 22.5 behind.

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

No branches or pull requests

3 participants