We can make a new issue about this, but it would be great to refactor from_pydict below to:
- Use
from_arrays internally like Table already does:
|
let (names, arrays): (Vec<_>, Vec<_>) = mapping.into_iter().unzip(); |
|
Self::from_arrays(cls, arrays, Some(names), schema, metadata) |
- Take in a
schema parameter
Originally posted by @kylebarron in #461 (comment)
We can make a new issue about this, but it would be great to refactor
from_pydictbelow to:from_arraysinternally like Table already does:arro3/pyo3-arrow/src/table.rs
Lines 335 to 336 in c8a9314
schemaparameterOriginally posted by @kylebarron in #461 (comment)