CTUAvastLab/JsonGrinder.jl

Remove automatic joining of scalars?

Closed this issue · 2 comments

So far scalar values are concatenated together for performance sake.
Since for most domains this occasion is very sparse, and causes inconsistency between structure of extractor and model, the performance gain, which is negligible for our usecases, is not worth the problems with changing structure.

The best idea would probably be to keep it as an option, which can be turned on, but it would be turned off by default.

it will be removed in JsonGrinder 2

I think it is removed by now