statnet / ndtv

ndtv: Network Dynamic Temporal Visualizations in R
https://cran.r-project.org/web/packages/ndtv/index.html
46 stars 5 forks source link

remove platform specific RStudio plot code #16

Open skyebend opened 8 years ago

skyebend commented 8 years ago

(transfered from statnet trac ticket #1189)

for history, see statnet trac ticket #301

sounds like RStudio may be releasing a work-around soon

On Sun, Jun 14, 2015 at 3:46 PM, Alex Chubaty <alex.chubaty@gmail.com> wrote:
> JJ, do you have an estimated timeframe for when this fix will appear?
> There's still the issue of RStudio users not updating their software at each
> and every release, so it may time some time for `dev.new` to work for most
> people.

This is in our daily builds now and will also be included in our next
minor patch release (within a few weeks).

J.J.

would be nice to remove this cran warning

comment:1 Changed 7 months ago by skyebend

I now see a new argument to dev.new(noRStudioGD = TRUE) however, this relies on Sys.getenv("R_DEFAULT_DEVICE") which is blank for me, so just creates pdf device. Usually not a good choice for animation.

Also, discussion on ​https://github.com/yihui/animation/issues/68

comment:2 Changed 7 months ago by skyebend

In 14082/statnet_commons:

modified the RStudio hack to use R_DEFAULT_DEVICE to avoid CMD check warning. References #1189