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

Widget in Dashboard didnt show the graph #1479

Closed
Knechtruprecht81 opened this Issue Jun 12, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@Knechtruprecht81

Knechtruprecht81 commented Jun 12, 2015

Hi,
the Problem is when i generate a chart

generatetchart1 1 2

and add this chart to dashboard the graph is not there.

generatechartindashboard

Graylog v1.1.2

@edmundoa

This comment has been minimized.

Member

edmundoa commented Jun 12, 2015

Hi,

Do you see any errors in your browser's Javascript console?

@Knechtruprecht81

This comment has been minimized.

Knechtruprecht81 commented Jun 12, 2015

No Errors:
generatetchart1 1 2_firebug

@edmundoa edmundoa added the bug label Jun 12, 2015

@edmundoa edmundoa added this to the 1.2.0 milestone Jun 12, 2015

@edmundoa edmundoa self-assigned this Jun 12, 2015

@edmundoa

This comment has been minimized.

Member

edmundoa commented Jun 12, 2015

I could reproduce the issue when the data coming from the server contains NaN. When Infinity and -Infinity are there, the resulting graph is also different than the one in the search result page, and sometimes a bit weird.

@Knechtruprecht81

This comment has been minimized.

Knechtruprecht81 commented Jun 12, 2015

Ok. Thank you.

@edmundoa edmundoa closed this in dfc3cff Jun 12, 2015

edmundoa added a commit that referenced this issue Jun 12, 2015

Normalize graph widget numbers before drawing them
Treat NaN, Ininity, and -Infinity as zero to avoid issues when graphing
them.

Fixes #1479

(cherry picked from commit dfc3cff)

@edmundoa edmundoa modified the milestones: 1.1.3, 1.2.0 Jun 12, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment