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

roam() does not work: object 'guess_type' not found #30

Closed
AlbertLei opened this issue Sep 30, 2024 · 2 comments
Closed

roam() does not work: object 'guess_type' not found #30

AlbertLei opened this issue Sep 30, 2024 · 2 comments

Comments

@AlbertLei
Copy link

AlbertLei commented Sep 30, 2024

following the previous issue #29:

after upgrading xfun as suggested, the browser is opened automatically after roam(). However, the preview does not work.

The viewer panel prints:

Error in is.factor(x) : object 'guess_type' not found

it happens in both Rstudio and Vscode.

> sessionInfo()
R version 4.4.0 (2024-04-24)
Platform: x86_64-apple-darwin20
Running under: macOS Sonoma 14.6.1

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib 
LAPACK: /Library/Frameworks/R.framework/Versions/4.4-x86_64/Resources/lib/libRlapack.dylib;  LAPACK version 3.12.0

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

time zone: Asia/Shanghai
tzcode source: internal

attached base packages:
[1] stats     graphics  grDevices utils     datasets 
[6] methods   base     

other attached packages:
[1] litedown_0.2.3

loaded via a namespace (and not attached):
[1] compiler_4.4.0 tools_4.4.0    xfun_0.47.6  

update: roam() works as expected after i downgrade xfun.

@AlbertLei
Copy link
Author

On a related note #29, i personally think print the url in the R console after roaming is worth considering.

As long as the link starts with https://, usually i can just click and open it in my web browser. That may make things easier for both u and users like me.

yihui added a commit to yihui/xfun that referenced this issue Sep 30, 2024
@yihui
Copy link
Owner

yihui commented Sep 30, 2024

Sorry, that's a bug. Fixed in xfun 0.47.7.

print the url in the R console after roaming is worth considering.

Done. Thanks for the suggestion!

clrpackages pushed a commit to clearlinux-pkgs/R-xfun that referenced this issue Oct 15, 2024
Arnaud Gallou (1):
      Remove `installed.packages()` usage and call `tools::standard_package_names()` instead (#91)

Yihui Xie (27):
      start the next version
      add .r and .rmd to the mimemap
      let [] return NULL instead of list()
      take advantage of the `bytes` argument of tools::md5sum() if available: wch/r-source@c91b845
      add a function mime_type(), which doesn't have to depend on the mime package, because MIME types could be obtained from system commands
      need to add System.Web before using the class
      it's also possible to call the `file` command on Windows (if Cygwin or Rtools is installed)
      if the mime type returned from command line doesn't appear to be valid, don't cache it but issue a warning instead
      use tools:::mime_type() if possible, and leave command-line as the last resort
      fix yihui/litedown#29: call browseURL() to open the app (getOptions('browser') is not a function in a normal R session outside RStudio)
      fix yihui/litedown#30: keep guess_type() until the next release of litedown
      follow the suggestion at yihui/litedown#30 (comment) to print the URL of the app (unless open = FALSE)
      partially revert 5557263bfd064efcfe7f241f094f8e5b73e427c8: base64_uri() no longer requires the mime package
      the mime package is no longer required
      add a method record_print.record_asis(), which is an identity function
      rename .ps1 to .txt to bypass the security scan on certain platforms (sigh...)
      add tests for .mime_cmd()
      make sure the powershell script is tested on windows
      not sure who is dumber
      revert fa1e0f15dec0f63bd648cf5afb50132691e2fe70: the URL is an implementation detail, and based on private API, so probably should be hidden from users by default; if they really want to see the URL, they should explicitly print it
      add new functions html_tag(), html_value(), html_escape(), and html_view() and remove dependency on htmltools
      add a function file_rename() for rstudio/bookdown#804
      add tests for file_rename()
      remove dependency on {markdown}, and switch to litedown
      url redirect
      just get rid of this URL since I trip on it every time I submit to CRAN
      CRAN release v0.48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants