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

fmt_number() not working with opt_interactive() #1942

Closed
laresbernardo opened this issue Jan 13, 2025 · 3 comments
Closed

fmt_number() not working with opt_interactive() #1942

laresbernardo opened this issue Jan 13, 2025 · 3 comments
Assignees

Comments

@laresbernardo
Copy link

Similar to issue #1429, fmt_number() is not working with opt_interactive(), displaying a white output with no table.

Reproducible example with gt v0.11.1:

gt::towny |>
  gt::gt() |>
  gt::fmt_number()

gt::towny |>
  gt::gt() |>
  gt::fmt_number() |>
  gt::opt_interactive()

Also, tested with v0.9.0, same outcome.

Funny thing is that in my local RStudio it is working as expected but in Posit Connect RStudio instance is not. Sharing my sessionInfo() from the Posit Connect's session:

R version 4.3.1 (2023-06-16)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Red Hat Enterprise Linux 8.10 (Ootpa)

Matrix products: default
BLAS/LAPACK: /shared-app/R-4.3.1/lib64/R/lib/libopenblas.so.0;  LAPACK version 3.9.0

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8   
 [6] LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C                  LC_ADDRESS=C               LC_TELEPHONE=C            
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C       

time zone: America/New_York
tzcode source: system (glibc)

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

other attached packages:
[1] gt_0.11.1

loaded via a namespace (and not attached):
 [1] vctrs_0.6.5       cli_3.6.3         rlang_1.1.4       generics_0.1.3    jsonlite_1.8.8    glue_1.8.0        htmltools_0.5.8.1 reactR_0.6.0     
 [9] sass_0.4.9        fansi_1.0.6       crosstalk_1.2.0   tibble_3.2.1      fastmap_1.2.0     yaml_2.3.10       lifecycle_1.0.4   compiler_4.3.1   
[17] dplyr_1.1.4       htmlwidgets_1.6.4 pkgconfig_2.0.3   rstudioapi_0.15.0 digest_0.6.35     R6_2.5.1          tidyselect_1.2.1  utf8_1.2.4       
[25] pillar_1.9.0      magrittr_2.0.3    tools_4.3.1       withr_3.0.0       reactable_0.4.4   xml2_1.3.6 
@olivroy
Copy link
Collaborator

olivroy commented Jan 13, 2025

Hi ! This actually is the same as #1835.

I seethat some of your packages are outdated, In this case, reactR 0.6.0 should be updated to 0.6.1.

@laresbernardo
Copy link
Author

laresbernardo commented Jan 14, 2025

Is there a way to add this requirement to the DESCRIPTION file, even though it's not a direct (import) dependency? I believe the path is gt > reactable> htmlwidgets > reactR. I guess adding reactR as a dependency with the >=0.6.1 requirement would be a viable solution, but maybe there's a better way.

@olivroy
Copy link
Collaborator

olivroy commented Jan 14, 2025

This would be a feature request for the reactable package (on which gt depends on)! Feel free to open a PR / issue there.

The path is gt -> reactable -> reactR

@olivroy olivroy closed this as not planned Won't fix, can't repro, duplicate, stale Jan 14, 2025
laresbernardo added a commit to laresbernardo/reactable that referenced this issue Jan 14, 2025
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