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

ETB behavior on 4chan #100

Closed
ElDominio opened this issue Feb 13, 2023 · 23 comments
Closed

ETB behavior on 4chan #100

ElDominio opened this issue Feb 13, 2023 · 23 comments
Labels

Comments

@ElDominio
Copy link
Collaborator

So I have had differing experiences with Hellen-based hardware on hellen154 vs 4chan

On Hyundai 154, I can use autotune just fine, and the PID settings are usually
P = 0xx.xx
I = 0xxx.xx
D = 0.xxxx

On 4chan
P = 0x.xx (usually around 0.8-2)
I = 0xx.xx (usually around 10-15)
D = 0.0xxx (usually around 0.02-0.09)
Using autotune drives it too crazy, and usually overheats. Using settings by hand usually makes it works quite well.

@rusefillc
Copy link
Contributor

@Qwerty-OFF any ideas why would Hellen-based Hyundai 154 use ETB PID settings matching Proteus while Hellen-based would be pretty different, with PID auto-tune working on Proteus and Hyundai 154 but failing on 4chan?

@mck1117
Copy link

mck1117 commented Feb 13, 2023

What sort of capacitors does hyundai board use near the ETB chip? Ceramic or electrolytic?

Proteus -> electrolytic

4chan -> ceramic

@rusefillc
Copy link
Contributor

old hyundai https://raw.githubusercontent.com/wiki/rusefi/rusefi/Hardware/Hellen/hellen154hyundai-rev-b.jpg ceramic?

new hyundai I do not have any photo :( @ElDominio ?

8chan ceramic

@Qwerty-OFF
Copy link
Collaborator

image

@rusefillc
Copy link
Contributor

@mck1117 adding electrolytic did not change anything :(

image

@rusefillc
Copy link
Contributor

@dron0gus says

image

image

@rusefillc
Copy link
Contributor

that did not help :(

WhatsApp Image 2023-03-16 at 16 21 34

WhatsApp Image 2023-03-16 at 16 30 23

@rusefillc
Copy link
Contributor

@mck1117 @andreika-git I am out of ideas. Sounds like we need an EE person to touch this? Would any sort of logs help do we even log useful stuff?

#109 created because why not

@rusefillc
Copy link
Contributor

#110 another crazy one

@mck1117
Copy link

mck1117 commented Mar 16, 2023

I don't think there are any useful logs to have here that aren't an oscilloscope.

@mck1117
Copy link

mck1117 commented Mar 16, 2023

actually, can you take a maxed out TS rate log of auto calibrate both in the GOOD and BAD scenarios, for clear side-by-side comparison?

@rusefillc
Copy link
Contributor

rusefillc commented Mar 16, 2023

@rusefillc
Copy link
Contributor

@mck1117 poke about above good vs bad

@andreika-git
Copy link
Collaborator

andreika-git commented Mar 24, 2023

The 4chan log has "OBD_TPS1_Correlation" error, which means the problem is with TPS1 (hardware or wrong settings?) - that's why PID autotune fails.

The OBD_TPS1_Correlation error appears when one of TPS sensors (TPS1 or TPS2) has an invalid signal. As a test, we can disable the TPS2 and try in 'ignoreSecond' mode.

@mck1117
Copy link

mck1117 commented Mar 24, 2023

bad left, good right

image

In the good case, it looks more sinusoidal, and in the bad case, it looks much steeper on opening than closing.

In the bad case there are also some weird "runt" cycles like this:
image

In the good case it's much more "stable"

@mck1117
Copy link

mck1117 commented Mar 24, 2023

The 4chan log has "OBD_TPS1_Correlation" error, which means the problem is with TPS1 (hardware or wrong settings?) - that's why PID autotune fails.

The OBD_TPS1_Correlation error appears when one of TPS sensors (TPS1 or TPS2) has an invalid signal. As a test, we can disable the TPS2 and try in 'ignoreSecond' mode.

That warning isn't active at the time autotune is running. That's not the problem.

@mck1117
Copy link

mck1117 commented Mar 24, 2023

that's why PID autotune fails.

The logs show the firmware is doing fine, and doing the same thing in both cases. It also looks like something is electrically or mechanically wrong: see the weird TPS waveform on the "bad" log.

@mck1117
Copy link

mck1117 commented Mar 24, 2023

Wait, alphax 4chan is using pin C6 for PWM.

stm32_common.cpp says

image

But mcuconf_common_f4_f7.h says

image

4CHAN IS USING SOFTWARE PWM FOR ETB

rusefillc added a commit to rusefi/rusefi that referenced this issue Mar 24, 2023
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Mar 24, 2023
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Mar 24, 2023
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Mar 25, 2023
rusefillc added a commit to rusefi/rusefi that referenced this issue Mar 25, 2023
rusefillc added a commit to rusefi/rusefi that referenced this issue Mar 25, 2023
rusefillc added a commit to rusefi/rusefi that referenced this issue Mar 25, 2023
rusefillc added a commit to rusefi/rusefi that referenced this issue Mar 25, 2023
@rusefillc
Copy link
Contributor

Fixed

Damn that's a shameful issue :(

see also rusefi/rusefi@367f586#diff-b3d072a3a06739829ae8497553994e0e8b74b4f24f3d23bfe96c69332d498822

@mck1117
Copy link

mck1117 commented Mar 27, 2023

Damn that's a shameful issue :(

this should be a lesson for “electrical weirdness -> get out a scope and look at it instead of shooting in the dark for a month”

@ElDominio
Copy link
Collaborator Author

@rusefillc was this tested before closing?

@rusefillc
Copy link
Contributor

@rusefillc was this tested before closing?

yes. my bad not posting happy artifacts

and this justifies a new release actually!

@ElDominio
Copy link
Collaborator Author

Tested today, way, way better results! :D

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

5 participants