SymbolixAU / jsonify

R package to convert R objects to JSON
https://symbolixau.github.io/jsonify/
Other
67 stars 10 forks source link

Prevent recursion on "json" objects #76

Open mmuurr opened 11 months ago

mmuurr commented 11 months ago

It's useful at times to include an already-JSON-encoded object in a new object for JSON-encoding, e.g. to prevent an unneeded decoding step (which can be expensive if performed (m|b|tr)-illions of times):

library(jsonify)
x_json <- read_json_object(from, somewhere)  ## ideally we want to not decode x_json
y_json <- list(foo = "bar", x = x_json) |> to_json()  ## re-encodes x_json as length-one string array :-|

This is doable with {jsonlite} provided:

  1. x_json has class "json"
  2. toJSON(...) includes the argument json_verbatim = TRUE.

This would be a useful addition to {jsonify} ... and I'm happy to take a stab at it with a PR if you're open to the idea.

dcooley commented 11 months ago

take a stab at it with a PR if you're open to the idea.

yeah go for it