Colin Jermain
Colin Jermain
PRs welcome!
@mivade, great suggestion! I agree that CSV files are not ideal for every case. The main advantage is that experiment parameters can easily be stored alongside the data in the...
@StePhanino, any luck addressing the review comments? It would be great to include this in the next release in a few weeks.
For `make venv`, on my machine I have to change to `python3 -m venv venv` to get the right version.
I'm interested in feedback on the function names. Here is a summary (omitting Option/Result as they are handled slightly differently in Py/Rs): - `json_infer(number_of_rows) -> DataType` - infers the Arrow...
One alternative to consider for the above function names is using `load` instead of `extract` => `json_load` and `json_path_load`. This is more consistent with the `json.loads` function in Python.
> as someone coming from a JS background, i've always found the python `loads` & `dumps` methods very unintuitive. I think `extract` or `parse` are much more self-explanitory Thanks. As...
> I'm wondering, what is the benefit of exposing the `json_infer` method? could we not do something similar to the read_* implementations. & take a schema or an infer_count >...
I'm getting back to this PR after some time. It needs updates in arrow2 and json_deserialize to fix jorgecarleitao/json-deserializer#13. There are also conflicts from the dispatch changes in #4423 that...
I've broken out the initial Rust functions into a different PR (#5140) to make this easier to review.