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

Three-phase energy calculation option #84

Merged
merged 2 commits into from
Mar 14, 2018

Conversation

glynhudson
Copy link
Contributor

@glynhudson glynhudson commented Mar 13, 2018

  • Added option to calculate 3-phase energy (multiple current by the square root of 3)..
  • Disabled by default

My notes on three-phase calculation:

- P(3ph) = V(LN) * I * 3
- V(VL) = 230V (line to neutral voltage)
- V(LL) = V(LN) * √3 = 230 * 1.75 = 400V

@glynhudson glynhudson changed the title Three-phase energy option Three-phase energy calculation option Mar 13, 2018
@lincomatic lincomatic merged commit b69abe2 into lincomatic:development Mar 14, 2018
@lincomatic
Copy link
Owner

thanks for the PR. Next time, please squash your PR's into one commit.

@glynhudson glynhudson deleted the 3-phase branch March 14, 2018 01:42
@beikeland
Copy link

Seems it got merged as * 3 instead of * sqrt(3) (or * 1.732 )

@lincomatic
Copy link
Owner

We have to keep the math as integer. Any comments, @glynhudson ?

@glynhudson
Copy link
Contributor Author

Sqrt(3) is only required when calculating power using Line-to-line voltage (usually 400V) when using Line-to-neutral voltage (usually 240V) multiplying the current by three is all that's required. See my notes on calculation in the PR above.

@beikeland
Copy link

beikeland commented Jun 28, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants