-
Notifications
You must be signed in to change notification settings - Fork 6
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
Handle UNDEFs #1
Comments
imaginator
added a commit
to imaginator/openhab2-prometheus-exporter
that referenced
this issue
May 14, 2019
Hmm yes. UNDEF is basically hard to display. What would be the right thing to do with the metric when it’s not defined? Simply leaving it out of the output is my first thought about this. Pretty sure that using a pre-defined number would be wrong. |
I agree with you - leave out.
I sent you a PR to just leave it out as I see you are already doing with
null values.
…On Tue, 14 May 2019, 21:06 Leroy Förster, ***@***.***> wrote:
Hmm yes. UNDEF is basically hard to display. What would be the right thing
to do with the metric when it’s not defined? Simply leaving it out of the
output is my first thought about this. Pretty sure that using a pre-defined
number would be wrong.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1?email_source=notifications&email_token=AADSURFV732SND6XLT2E2XDPVMETDA5CNFSM4HMYQKG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVMPG5Y#issuecomment-492368759>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADSURHEE23SYAXN7SDLDYDPVMETDANCNFSM4HMYQKGQ>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is exactly what I was looking for. Thanks.
Small issue: some of my values are
UNDEF
:Which leads to this:
The text was updated successfully, but these errors were encountered: