Skip to content
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

add support for time series #4

Open
timelyportfolio opened this issue Jan 5, 2015 · 4 comments
Open

add support for time series #4

timelyportfolio opened this issue Jan 5, 2015 · 4 comments
Assignees

Comments

@timelyportfolio
Copy link

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 rstudio/dygraphs#16 and date formats rstudio/dygraphs#21 .

Happy to close if you think not helpful.

@hrbrmstr
Copy link
Owner

hrbrmstr commented Jan 5, 2015

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
Copy link
Author

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

@hrbrmstr hrbrmstr self-assigned this Jan 5, 2015
@hrbrmstr
Copy link
Owner

hrbrmstr commented Jan 6, 2015

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
Copy link
Contributor

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
#4 (comment)
.

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

No branches or pull requests

3 participants