Warning: partial argument match of 'file' to 'filename'

[bersbersbers]

options(warnPartialMatchArgs = TRUE)
library(ggplot2)
library(plotly, warn.conflicts = FALSE)
ggplotly(ggplot(mtcars, aes(wt, mpg)) + geom_point())

issues

Warning message:
In dev_fun(file = tempfile(), width = width %||% 640, height = height %||%  :
  partial argument match of 'file' to 'filename'

https://github.com/ropensci/plotly/blob/bdfdc4b15e6c3cb389384b422cfed7355dcce202/R/ggplotly.R#L179-L200

Related: tidyverse/ggplot2@93882e6

[bersbersbers]

Using options(showWarnCalls = TRUE), I have been able to get this extra detail:

In dev_fun(file = tempfile(), width = width %||% 640, height = height %||%  :
  partial argument match of 'file' to 'filename'
Calls: <Anonymous> ... eval -> eval -> <Anonymous> -> ggplotly.ggplot -> gg2list

confirming the relevance of the above code snippet.

[bersbersbers]

I tried out a solution in https://github.com/bersbersbers/plotly/tree/fix-1977 and it seems to fix the warnings.

[bersbersbers]

Since that fork is gone, for the record:

diff --git a/R/ggplotly.R b/R/ggplotly.R
index e29dd2b9..645d5018 100644
--- a/R/ggplotly.R
+++ b/R/ggplotly.R
@@ -181,7 +181,7 @@ gg2list <- function(p, width = NULL, height = NULL,
   } else if (capabilities("jpeg")) {
     grDevices::jpeg 
   } else if (system.file(package = "Cairo") != "") {
-    Cairo::Cairo
+    function(filename, ...) Cairo::Cairo(file = filename, ...)
   } else {
     stop(
       "No Cairo or bitmap device is available. Such a graphics device is required to convert sizes correctly in ggplotly().\n\n", 
@@ -197,7 +197,7 @@ gg2list <- function(p, width = NULL, height = NULL,
     height <- height %||% default(grDevices::dev.size("px")[2])
   }
   # open the device and make sure it closes on exit
-  dev_fun(file = tempfile(), width = width %||% 640, height = height %||% 480)
+  dev_fun(filename = tempfile(), width = width %||% 640, height = height %||% 480)
   on.exit(grDevices::dev.off(), add = TRUE)
   
   # check the value of dynamicTicks

[moutikabdessabour]

Hello @bersbersbers, thank you for reporting this. Are you interested in making a PR?

[bersbersbers]

@moutikabdessabour sure! #2046