Add UPS Power (in Watts) to the UPS Status page #339
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This will differ from the current UPS Load percentage, as that is calculated in VA, not Watts
|
Before this pull request can be accepted you must first sign a CLA as described at https://www.pfsense.org/about-pfsense/#cla. Please read for more details. |
|
UPS power is already displayed on this page in the UPS Detail section (assuming the UPS provides it). Scroll down and you should see it. FWIW, the reason UPS power is not in the top section is because a) it's rare compared to UPS load which is almost universal, and b) it's not used as part of the runtime calculation. The top section and the widget contain only those items pertaining to failure (quality and runtime). |
|
Fair enough |
netgate-git-updates
pushed a commit
that referenced
this pull request
Sep 19, 2020
* Provide --version option (#372) * Version information now within generated certificates like on *nix * Fixed issue where gen-dh overwrote existing files without warning (#373) * Fixed issue with ED/EC certificates were still signed by RSA (#374) * Added support for export-p8 (#339) * Clarified error message (#384) * 2->3 upgrade now errors and prints message when vars isn't found (#377)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As the title suggests - this adds a new row that shows how many Watts are being drawn from the UPS.
This will differ from the current UPS Load percentage, as that is calculated in VA, not Watts. I'm open to discussion whether real power (W) or apparent power should be used.