-
Notifications
You must be signed in to change notification settings - Fork 2
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
Charge Control C: add section regarding phase count switching support #29
base: everest/charge_control_c
Are you sure you want to change the base?
Charge Control C: add section regarding phase count switching support #29
Conversation
Signed-off-by: Michael Heimpold <michael.heimpold@chargebyte.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise lgtm, I actually don't see the diff in most pictures here, but that also could be the pr-review view. Have all those images changed?
No, I just adjusted the permissions. |
d2ccd0d
to
9199262
Compare
* **none** (default): No phase-count switching is supported - only R1/S1 is used to switch on/off a single contactor. | ||
* **serial**: Phase-count switching support is enabled using the serial wiring as described above: R1/S1 switches | ||
the primary contactor, R2/S2 is attached to the secondary contactor. | ||
* **mutual**: Phase-count switching support is enabled using the mutual wiring as described above. R1/S1 is wired to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"mutual" is currently not supported or? We should mention it in the documentation.
Signed-off-by: Michael Heimpold <michael.heimpold@chargebyte.com>
9199262
to
f8654be
Compare
No description provided.