search

paulklemm / rvisidata

Open any dataframe with visidata, a terminal-based spreadsheet application
Other
30 stars 2 forks source link

Optionally prefer to use jsonlite for serialization over write.csv #5

Closed strazto closed 4 years ago

strazto commented 4 years ago

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 .

paulklemm commented 4 years ago

Great contribution, merged PR into master.