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

launch control: Speed threshold depends on launch mode, also zero speed limit handling #5419

Closed
rusefillc opened this issue Jul 20, 2023 · 2 comments

Comments

@rusefillc
Copy link
Contributor

@Gorfest-n reports
RusEFI master.2023.07.10.alphax-4chan.1914450217

Speed threshold does not work in launch control.
Speed threshold is set to 10 km/h, but after reaching 10 km/h the launch control does not turn off. Red line - turn on the ignition cutoff while reaching the launch rpm.

@rusefillc
Copy link
Contributor Author

launchSpeedThreshold

@rusefillc
Copy link
Contributor Author

@Gorfest-n code says

/**
 * Returns True in case Vehicle speed is less then threshold.
 * This condition would only return true based on speed if DisablebySpeed is true
 * The condition logic is written in that way, that if we do not use disable by speed
 * then we have to return true, and trust that we would disable by other condition!
 */ 
bool LaunchControlBase::isInsideSpeedCondition() const {
	int speed = Sensor::getOrZero(SensorType::VehicleSpeed);
	
	return (engineConfiguration->launchSpeedThreshold > speed) || (!(engineConfiguration->launchActivationMode ==  ALWAYS_ACTIVE_LAUNCH));
}

which is why it's nice to have a link to your current tune to know your launchActivationMode

@rusefillc rusefillc changed the title Speed threshold does not work in launch control launch control: Speed threshold depends on launch mode, also zero speed limit handling Jul 20, 2023
rusefillc added a commit that referenced this issue Jul 20, 2023
rusefillc added a commit that referenced this issue Jul 20, 2023
…ed limit handling #5419

only: change zero setting handling
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

No branches or pull requests

1 participant