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

Reversed field direction in SINE variant #40

Closed
jsphuebner opened this issue Oct 24, 2023 · 3 comments
Closed

Reversed field direction in SINE variant #40

jsphuebner opened this issue Oct 24, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@jsphuebner
Copy link
Owner

Sine firmware seems to generate a stator field rotating anti-clockwise when forward is selected and clockwise otherwise.
It has been discovered by catphish but I can't find the forum post.

Please help searching

As a consequence people using the LDU need to swap the A & B encoder wires and select reverse direction when wanting to go forward. Same goes for the SDU but there the swap is already incorporated into the hardware.
Also the newly introduced "rotordir" spot value indicates this phase swap: https://openinverter.org/forum/viewtopic.php?p=62563#p62563

When fixing this we need to keep backward compatibility in mind. People who use an A/B swapped LDU or SDU shouldn't need to touch their wiring.

Perhaps it's as simple as introducing "pinswap" in sine firmware as well, allowing to swap PWM channels 1 and 2. Encoder channels can't be swapped in software.

@jsphuebner jsphuebner added the bug Something isn't working label Oct 24, 2023
@Mitchy6
Copy link

Mitchy6 commented Oct 25, 2023

Is this something relating to the issue?
https://openinverter.org/forum/viewtopic.php?p=52723#p52723

May also be related to the Current CT Swap?
https://openinverter.org/forum/viewtopic.php?p=50975#p50975

@jsphuebner
Copy link
Owner Author

Yes that's the post I was looking for, thanks.

Swapped current sensors are not relevant in sine firmware

@jsphuebner
Copy link
Owner Author

Added pinswap to sine firmware 730dfa3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants