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

[windows] change default datatype for pdh queries to double. Integer #3703

Merged
merged 2 commits into from
Mar 15, 2018

Conversation

derekwbrown
Copy link
Contributor

loses fidelity, especially with values between 0 and 1.

What does this PR do?

change default pdh counter precision to float from int.
Allows configuration to set specific counter types

Motivation

customer request

loses fidelity, especially with values between 0 and 1.
Copy link
Member

@truthbk truthbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we might have an import issue when we make the imports from the stub file. Could you please double-check? 🙇

except ImportError:
from checks.libs.win.winpdh_stub import WinPDHCounter
from checks.libs.win.winpdh_stub import WinPDHCounter,DATA_TYPE_INT, DATA_TYPE_DOUBLE
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think DATA_TYPE_INT, DATA_TYPE_DOUBLE has been defined in the stub.

Copy link
Member

@truthbk truthbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully we can have a single source of truth soon, this is annoying to maintain! Thank you! 🙇

@derekwbrown derekwbrown merged commit eec3773 into master Mar 15, 2018
@derekwbrown derekwbrown deleted the db/pdhfloat branch March 15, 2018 18:44
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.

None yet

2 participants