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.

Sign up for GitHub

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

Jump to bottom

turbo speed turbospeed input #2935

Open
rusefillc opened this issue Jul 9, 2021 · 7 comments
Open

turbo speed turbospeed input #2935

rusefillc opened this issue Jul 9, 2021 · 7 comments
Assignees
@AlxrMironovOldAcc

Comments

@rusefillc
Copy link
Contributor

rusefillc commented Jul 9, 2021 •
edited

120krpm
12compressor blades, sensor divide by factor of 8

if you theoretically have 8blade compressor you will have 1pulse per revolution 120k pulses per minute ie 2000hz

250'000 RPM is 4166 Hz

at 1.5 event per revolution that's 6250Hz

that's pretty much same as 60 tooth at 8000 RPM

Do we want to input as is or divide a bit?
@rusefillc
Copy link
Contributor Author

andrey 2 hours ago
120krpm max, what is min?
New

Simon - Orchardperformance 21 minutes ago
Pretty much zero

Simon - Orchardperformance 21 minutes ago
Idling speed of the turbo can be sub 10k

Martin Matousek 16 minutes ago
aalso jacques sensor output is low (no pulses) below 5krpm or something, it probably allso have internal pullup to 5v (edited)

Martin Matousek 15 minutes ago
and yes during idle turbospeed sensor can show 0krpm for sure and for smaller turbos 190 maybe even 200,210,220krpm is possible (edited)

Simon - Orchardperformance < 1 minute ago
Ours show a flicker from 1.5k to 2k.
Its pretty much flatline/ signal noise

Simon - Orchardperformance < 1 minute ago
250k is right up at the top end of anything small available right now

@rusefillc rusefillc changed the title turbospeed input turbo speed turbospeed input Jul 27, 2021
@rusefillc
Copy link
Contributor Author

Related or not to rusefi/hellen76nissan-issues#3?

@rusefillc
Copy link
Contributor Author

https://www.full-race.com/docs/Speed-Sensor-Installation-Instructions.pdf

@mck1117
Copy link
Member

mck1117 commented Jul 27, 2021

250k rpm * 14 blades / 8 = 7.3khz

https://www.wolframalpha.com/input/?i=250000%2Fmin+*+14+%2F+8

@mck1117
Copy link
Member

mck1117 commented Jul 27, 2021

an additional divide-by-8 would give us sub-1khz worst case

@mck1117
Copy link
Member

mck1117 commented Jul 27, 2021

divide by 2 is easy with a D flip flop - chain more to divide by 2^n:

image

rusefillc pushed a commit that referenced this issue Jul 27, 2021
turbo speed turbospeed input #2935
3f36e73
@rusefillc
Copy link
Contributor Author

Configuration fields added, next step is deciding if we copy/paste vehicle_speed.cpp or reuse vehicle_speed or else

See also - kind of depends on #3081

rusefillc added a commit that referenced this issue Sep 20, 2021
@AlxrMironovOldAcc @rusefillc
Turbo speed sensor implementation #2935 (#3209)
bb341c4 
* Add turbocharger speed converter class

* Add turbocharger speed converter tests

* Add turbocharger speed sensor initialization functions

* Add turbocharger speed sensor to initialization

Co-authored-by: alxrMironov <330OMcorporative>
Co-authored-by: rusefillc <48498823+rusefillc@users.noreply.github.com>
@rusefillc rusefillc mentioned this issue Oct 26, 2021
Closed
rusefillc added a commit that referenced this issue Nov 4, 2021
@rusefillc
turbo speed turbospeed input #2935
433d526
rusefillc pushed a commit that referenced this issue Nov 5, 2021
turbo speed turbospeed input #2935
e8d6f30
rusefillc pushed a commit that referenced this issue Nov 5, 2021
turbo speed turbospeed input #2935
8f2bd93
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlxrMironovOldAcc AlxrMironovOldAcc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mck1117 @AlxrMironovOldAcc @rusefillc