paulklemm / rvisidata Goto Github PK

Just wanted to say I appreciate this library. Thanks, and keep up the good work!

Hi -
I work in a lot of remote ssh sessions, usually from the R console, and I'm interested in exploring visidata for a nicer way to view data.
I'd like to use your package, but I do wonder if the dependencies on magrittr and readr are necessary, given the minimalism of the codebase.

I also enjoy magrittr syntax, and the readr interface over base R, but for a package this small, I wonder if we could tolerate eliminating all non-base dependencies and forgoing the pipe? Thanks :)

Hi - thanks for making this package! I'm trying to give it a go, wondering if you know what might be causing this.

> library(rvisidata)
> rvisidata::vd(iris)
Loading required package: jsonlite
Warning message:
In system2(vd_cmd, args = temp_file) : '"vd"' not found

I suspect it's related to the below and my install of visidata. I can open visidata by typing "visidata" in the command line, but not through the vd filename.csv method on Windows.

>vd --version
'vd' is not recognized as an internal or external command,
operable program or batch file.

Thanks for any help on this!

Hi, I just found visidata and this amazing package when I was frustrated with excel. Visidata itself works well to me, but I tried the example of this package, it doesn't work somehow.

Steps

1. Open R with R --vanilla, everything in my .visdatarc are commented out.
2. rvisidata::vd(iris)

It gives

> sessionInfo()
R version 4.0.3 (2020-10-10)
Platform: x86_64-apple-darwin17.0 (64-bit)
Running under: macOS Big Sur 10.16

Matrix products: default
BLAS:   /Library/Frameworks/R.framework/Versions/4.0/Resources/lib/libRblas.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/4.0/Resources/lib/libRlapack.dylib

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

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

loaded via a namespace (and not attached):
[1] compiler_4.0.3  jsonlite_1.7.2  rvisidata_0.0.3

Hi,
I'd like to use jsonlite::write_json for serializing dataframes over the older, less flexible and less standardized csv.

My motivation is that there's write.csv is unable to serialize list columns and other more complex datastructures

I think it could be made an optional dependency, allowing a fallback to base R if the package isn't available.

I've implemented it on a branch on my fork, but it's built on another upstream branch that adds tmux integration ( #4 ), so I'll put in the PR, but that's under the assumption that you accept my first PR.

I can base this on your master if you reject #4 .

Hi,
I use tmux a lot in my terminal workflows, and a nice tmux setup can really help the terminal feel more comfortable than an IDE.

I wonder if you'd be interested in adding some optional tmux integration to this package?

Something along the lines of:

• if options(rvisidata.use_tmux) is true
  • Detect if we are currently in a tmux session

    Sys.getenv("TMUX")
    #> [1] "/tmp/tmux-1000/default,31069,1"

  • If not, use the standard vd command
    • Otherwise:
      • Split to a new pane in the tmux window
      • Run vd on our temporary csv in that pane
      • Return focus to the R pane (? probably most useful)

Scripting tmux to perform a sequence of commands is not too hard.

I guess that script could go in inst, to provide a decent default tmux environment for viewing the data.

There's not really a single "best" workspace for mixing between data exploration and interactive console use, so I imagine that in some instances it will be more useful to open vd in a new tmux window, etc.

If that's the case, maybe allowing users to swap in their own scripts may helpful at some stage.

I'll fork and PR for now, and let me know what you think!

Tried solving it using this but it doesn't work. Full details:

> devtools::install_github('paulklemm/rvisidata')
 26 Downloading GitHub repo paulklemm/rvisidata@master
 27 tar: Failed to set default locale
 28 tar: Failed to set default locale
 29    During startup - Warning messages:
 30    1: Setting LC_TIME failed, using "C"
 31    2: Setting LC_MESSAGES failed, using "C"
 32    3: Setting LC_MONETARY failed, using "C"
 33    checking for file ‘/private/var/folders/8w/w1h6bnd123s30145p0h6lnq00000gn/T/RtmpmlX3tZ/remotes1508a52b115e7/paulklemm-rvisidata-119f0a6/DESCRIPTION’ ..
 34 ✔  checking for file ‘/private/var/folders/8w/w1h6bnd123s30145p0h6lnq00000gn/T/RtmpmlX3tZ/remotes1508a52b115e7/paulklemm-rvisidata-119f0a6/DESCRIPTION’
 35 ─  preparing ‘rvisidata’:
 36 ✔  checking DESCRIPTION meta-information ...
 37 ─  checking for LF line-endings in source and make files and shell scripts
 38 ─  checking for empty or unneeded directories
 39 ─  building ‘rvisidata_0.0.3.tar.gz’
40
 41 Error: (converted from warning) Setting LC_TIME failed, using "C"
 42 Execution halted
 43 Error: Failed to install 'rvisidata' from GitHub:
 44   (converted from warning) installation of package ‘/var/folders/8w/w1h6bnd123s30145p0h6lnq00000gn/T//RtmpmlX3tZ/file1508a68151b1a/rvisidata_0.0.3.tar.gz’
 45  had non-zero exit status

I'm running NVIM on macOS Catalina Ver 10.15.5.

:ver
NVIM v0.4.3
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedant
ic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdia
gnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/tmp/neovim-20191107-85913-1wbgqz6/n
eovim-0.4.3/build/config -I/tmp/neovim-20191107-85913-1wbgqz6/neovim-0.4.3/src -I/usr/local/include -I/tmp/neovim-20191107-85913-1wbgqz6/neovim-0.4.3/deps-bui
ld/include -I/usr/local/opt/gettext/include -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/usr/include
-I/tmp/neovim-20191107-85913-1wbgqz6/neovim-0.4.3/build/src/nvim/auto -I/tmp/neovim-20191107-85913-1wbgqz6/neovim-0.4.3/build/include
Compiled by [email protected]

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.4.3/share/nvim"

I have installed visidata as per this using brew install saulpw/vd/visidata, vd --version prints saul.pw/VisiData v1.5.2. I am attempting to install this from within Nvim-R.

Edit: The issue was bypassed by installing rvisidata through RStudio. After doing that, opening Nvim-R, loading the package and opening up the mtcars dataset worked.