bar charts #113

Closed
wants to merge 7 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

piotr1212 commented Nov 30, 2012

Hi, I've added graphType=bar to Graphite on request of my collegues. I thought it would be nice to share. Please let me know what you think,

Regards, Piotr

Owner

mleinart commented Feb 9, 2013

This is pretty neat, if someone would build on this further I think it'd be a good addition. A few things that would make this more complete:

  • support mixed negative and positive numbers
  • use drawRectangle() and fill the bars rather than doing the trace manually
  • make barWidth or barSpacing a configurable value (perhaps as a max or min rather than a constant)
  • draw multiple series by laying bars beside each other
    • This may entail making a special case for xStep calculation/series consolidation
  • support stacked bars
    • This would need different rendering logic from the normal stacked mode

I started on 1 and 2 here mleinart/graphite-web@9ed5669 but probably wont get around to getting it fixed up myself

Contributor

piotr1212 commented Dec 21, 2015

I won't be finishing this, Grafana does this way better than I could ever built in Graphite.

@piotr1212 piotr1212 closed this Dec 21, 2015

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