search

hrbrmstr / metricsgraphics

:chart_with_upwards_trend: htmlwidget interface to the MetricsGraphics.js D3 chart library
http://hrbrmstr.github.io/metricsgraphics/
Other
132 stars 35 forks source link

add support for time series #4

Open timelyportfolio opened 9 years ago

timelyportfolio commented 9 years ago

As you can probably tell, I am very interested in this. If ok, can we add this issue to track progress on the quote from Readme.md?

does not really take advantage of metricsgraphics' best feature - time series charts

@jjallaire might have some ideas here and think it would be very good to leverage the infrastructure with xts provided by rstudio/dygraphs lines especially given some of the complexities already discovered with things such as time zones https://github.com/rstudio/dygraphs/pull/16 and date formats https://github.com/rstudio/dygraphs/issues/21 .

Happy to close if you think not helpful.

hrbrmstr commented 9 years ago

Def keep the issue. I was going to "borrow" much of the xts goodness from the widget-ized dygraphs. I actually want to add far more intelligence into the whole pkg. Pick better defaults for size based on chart type (and data size). Auto-assign decent colors, etc.

timelyportfolio commented 9 years ago

do you want to add an issue for these additional features or maybe add a Todo.md?

hrbrmstr commented 9 years ago

basic time series support is in there now. it relies on setting mjs_axis_x(xax_format="date"). i'm keeping this open until i get support in for xts, etc.

jjallaire commented 9 years ago

This is shaping up very, very nicely! So excited to see this come together. BTW feel free to take whatever you need of the xts code in dygraphs :-)

On Mon, Jan 5, 2015 at 9:37 PM, boB Rudis notifications@github.com wrote:

basic time series support is in there now. it relies on setting mjs_axis_x(xax_format="date"). i'm keeping this open until i get support in for xts, etc.

— Reply to this email directly or view it on GitHub https://github.com/hrbrmstr/metricsgraphics/issues/4#issuecomment-68818080 .