NaN values being saved in Whisper datafiles

Bug #867882 reported by Nicholas Leskiw
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Graphite
Fix Released
Undecided
Unassigned

Bug Description

NaN values are somehow getting stored in datafiles. They should be converted to None as they're being read into carbon-cache.py, or at minimum, glyph.py should be configured to convert them to None. Otherwise Cairo can't render the graph properly. rawData can still return the data properly.

Revision history for this message
chrismd (chrismd) wrote :

There was an explicit check for NaNs in the MetricPickleReceiver class but not in the MetricLineReceiver (or MetricDatagramReceiver) for that matter. I've fixed this by moving it to the base MetricReceiver class.

Changed in graphite:
status: New → Fix Committed
Revision history for this message
Corry Haines (tabletcorry) wrote :

Wow, that's service :)

Thanks!

Revision history for this message
Corry Haines (tabletcorry) wrote :

Will there be a fix committed to allow the render engine to filter these as well, or do I need to cleanse my datastores of the NaN values?

Revision history for this message
chrismd (chrismd) wrote :

Just committed that as well

Revision history for this message
Nicholas Leskiw (nleskiw) wrote :

Heh, I just posted that in the question. You need to cleanse the datastore by sending a 0 or None to the corresponding time of each nan, or completely erase the .wsp file and start over.

Revision history for this message
Corry Haines (tabletcorry) wrote :

Indeed, duplication.

I will do the "send 0" thing to all of the NaNs, should be pretty easy.

Revision history for this message
Michael Leinartas (mleinartas) wrote :

Released in 0.9.9

Changed in graphite:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.