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 bug? "range" thing is confusing #4348

Closed
elmepro opened this issue Jul 15, 2022 · 9 comments
Closed

Launch control bug? "range" thing is confusing #4348

elmepro opened this issue Jul 15, 2022 · 9 comments
Labels

Comments

@elmepro
Copy link

elmepro commented Jul 15, 2022

If launch control is set to 2000rpm, then in real world i got 4000rpm looks like its two times more rpm than set to TunerStudio.
(separate request edited out by @rusefillc )

Car bmw m50b25 tigger wheel 60/2
Board: Proteus
FW: 20220702

If something needed let me known.

@shadowm60
Copy link
Collaborator

Can you share tune?

@elmepro
Copy link
Author

elmepro commented Jul 15, 2022

@rusefillc
Copy link
Contributor

please share log of launch attempt as well

@rusefillc
Copy link
Contributor

@elmepro I have to request that one github issue coveres only well, one subject

@rusefillc rusefillc added the bug label Jul 16, 2022
@elmepro
Copy link
Author

elmepro commented Jul 17, 2022

Im sorry for late response a had 2 day drifting training. (absolutely love rusefi)

Tune is same as uploaded here are log file
launch control log.zip

@rusefillc
Copy link
Contributor

@elmepro you say driftning on rusEFI and no photos/no videos? do you want to break my heart?

@elmepro
Copy link
Author

elmepro commented Jul 20, 2022

@elmepro you say driftning on rusEFI and no photos/no videos? do you want to break my heart?

Ouh don't worry I'm all in to Rusefi need small touches to car wiring and i will add videos and review

@rusefillc rusefillc changed the title Launch control bug? Launch control bug? "range" thing is confusing Jul 20, 2022
rusefillc pushed a commit that referenced this issue Jul 20, 2022
rusefillc pushed a commit that referenced this issue Jul 20, 2022
refactoring: reducing confusion
rusefillc pushed a commit that referenced this issue Jul 20, 2022
rusefillc pushed a commit that referenced this issue Jul 20, 2022
bug or dead code? at least making it a bit more obvious
@rusefillc
Copy link
Contributor

@elmepro it's a usability problem not a bug here

See https://rusefi.com/forum/viewtopic.php?p=45889#p45889

@rusefillc
Copy link
Contributor

will replace "range" with "adder" on the dialog labels

rusefillc pushed a commit that referenced this issue Jul 20, 2022
a typo and configurable behaviour without UI
rusefillc pushed a commit that referenced this issue Jul 21, 2022
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