rstudio / dygraphs

R interface to dygraphs
http://rstudio.github.io/dygraphs
Other
364 stars 193 forks source link

submit to CRAN? #164

Closed timelyportfolio closed 6 years ago

timelyportfolio commented 7 years ago

dygraphs on CRAN is missing all the fantastic changes by @pshevtsov since January 4, 2017. Would it be possible to submit an updated version to CRAN? Thanks.

przmv commented 7 years ago

Let me finish with https://github.com/rstudio/dygraphs/pull/157 and then we're good to update. What do you think, @jjallaire ?

jjallaire commented 7 years ago

Sounds good! Let me know when #157 is ready to go.

przmv commented 7 years ago

Sure thing!

On Thu, May 18, 2017 at 4:38 PM JJ Allaire notifications@github.com wrote:

Sounds good! Let me know when #157 https://github.com/rstudio/dygraphs/pull/157 is ready to go.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rstudio/dygraphs/issues/164#issuecomment-302406154, or mute the thread https://github.com/notifications/unsubscribe-auth/AARukzwpmeWxDd11BdcLxwObnmf-3oXBks5r7EnsgaJpZM4NfEJs .

timelyportfolio commented 7 years ago

Happy to help @pshevtsov, so just let me know. Using dygraphs very extensively on a current project so very engaged.

przmv commented 6 years ago

@timelyportfolio could you please help me with #157?

timelyportfolio commented 6 years ago

@pshevtsov, sure what needs to be done? Happy to help.

przmv commented 6 years ago

@timelyportfolio, I want you to try dyArrow with different data sets and make sure everything works OK. I'm going to write some documentation on how to style popups with CSS later today.

odelaigue commented 6 years ago

@pshevtsov, I tested dyArrow with different data (different time step, regular or not).

It seems that it does not work when the data used in the dygraph function is a data.frame and not a ts or a xts object.

library(xts) data(sample_matrix) dygraph(sample_matrix) %>% dyArrow("2007-1-29", "test 1", direction = "down", fillColor = "red") %>% dyArrow("2007-2-1", "test 2", direction = "down", fillColor = "red")

When the date of the event does not exist in the dataset, there is no arrow, but also no warning. And if I am right, it is the case in your example of the dyArrow function.

dygraph(presidents, main = "Quarterly Presidential Approval Ratings") %>% dyArrow("1950-6-30", "Korea", direction = "down", fillColor = "red") %>% dyArrow("1965-2-09", "Vietnam", direction = "down", fillColor = "red")

Another aspect is that the arrow is drawn on the last variable used (the last column fo example), maybe it will be usefull if it will be possible to give a name argument to choose more directly the curve.

lungDeaths <- cbind(fdeaths, mdeaths, ldeaths, foo = fdeaths/2, bar = fdeaths/3) dygraph(lungDeaths) %>% dyArrow("1977-6-01", "test 1", direction = "down", fillColor = "red") %>% dyArrow("1979-2-01", "test 2", direction = "down", fillColor = "red") %>% dyOptions(animatedZooms = TRUE)

przmv commented 6 years ago

Thanks @odelaigue, I'll have a look this weekend.

thodoha commented 6 years ago

Hi, can I check when will the updated version of dygraphs is submitted to CRAN? Or in other cases, how can I download the package from github?

jjallaire commented 6 years ago

You can install from GitHub with devtools::install_github("rstudio/dygraphs")

jjallaire commented 6 years ago

@pshevtsov Could you submit the version with all the recent changes to CRAN?

klaukh commented 6 years ago

Is it possible to take another look at #179 before CRAN? Requested changes were made and it's handy to define only a single color or single series color rather than the whole.

Also have been using extensively at project for awhile... it's awesome to pass a saved widget to another analyst that let's then explore rather than just a static view

przmv commented 6 years ago

@jjallaire, I've stuck a bit with that dyArrow implementation. I discovered an issue, which also affects dyAnnotation (thanks to @odelaigue for providing the data set to reproduce this issue). Sure thing, let's submit to CRAN what currently is in master branch.

@klaukh, I'm going to check PR #179 right now. If everything is fine, I'll merge the pull request before submitting to CRAN.

przmv commented 6 years ago

Dear maintainer,

package dygraphs_1.1.1.5.tar.gz has been auto-processed and is pending a manual inspection. A CRAN team member will typically respond to you within the next 5 working days.

przmv commented 6 years ago

dygraphs 1.1.1.5 is on CRAN — https://cran.rstudio.com/web/packages/dygraphs/index.html

odelaigue commented 6 years ago

@pshevtsov, thank you!

jjallaire commented 6 years ago

Terrific! @pshevtsov so happy that all of your great work is now on CRAN!

przmv commented 6 years ago

Thanks, @jjallaire! It's only the beginning!