fix: Fix Generex load sensor divisor #6155

Merged
merged 1 commit into from Mar 9, 2017

Conversation

Projects
None yet
5 participants
@FTBZ
Contributor

FTBZ commented Mar 9, 2017

  • Have you signed the Contributors agreement - please do NOT submit a pull request unless you have (signing the agreement in the same pull request is fine). Your commit message for signing the agreement must appear as per the docs.
  • Have you followed our code guidelines?

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926


screen shot 2017-03-09 at 14 13 00

  • As confirmed by many members, the load sensor seems to be using a wrong divisor on with Generex card

Details: #5977

@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot Mar 9, 2017

Thank you for submitting a PR @FTBZ! We have found the following @HenocKA, @murrant and @laf based on the history of these files to review this PR.

Thank you for submitting a PR @FTBZ! We have found the following @HenocKA, @murrant and @laf based on the history of these files to review this PR.

@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

scrutinizer-notifier Mar 9, 2017

The inspection completed: No new issues

The inspection completed: No new issues

@murrant murrant merged commit 6f2ba2b into librenms:master Mar 9, 2017

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment