Skip to content
This repository was archived by the owner on Dec 1, 2025. It is now read-only.

Commit 60c93da

Browse files
authored
Merge pull request #83 from lincc-frameworks/from_lists-meta
Fix meta derivation for from_lists
2 parents e1e1993 + 67c36e7 commit 60c93da

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/nested_dask/core.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
import pyarrow as pa
1010
from dask.dataframe.dask_expr._collection import new_collection
1111
from nested_pandas.series.dtype import NestedDtype
12-
from nested_pandas.series.packer import pack, pack_flat
12+
from nested_pandas.series.packer import pack, pack_flat, pack_lists
1313
from pandas._libs import lib
1414
from pandas._typing import AnyAll, Axis, IndexLabel
1515
from pandas.api.extensions import no_default
@@ -414,7 +414,7 @@ def from_lists(cls, df, base_columns=None, list_columns=None, name="nested"):
414414

415415
meta = npd.NestedFrame(df[base_columns]._meta)
416416

417-
nested_meta = pack(df[list_columns]._meta, name)
417+
nested_meta = pack_lists(df[list_columns]._meta, name)
418418
meta = meta.join(nested_meta)
419419

420420
return df.map_partitions(

tests/nested_dask/test_nestedframe.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
import pytest
99
from nested_dask.datasets import generate_data
1010
from nested_pandas.series.dtype import NestedDtype
11+
from pandas.testing import assert_frame_equal
1112

1213
dask.config.set({"dataframe.convert-string": False})
1314

@@ -205,6 +206,7 @@ def test_from_lists():
205206
ndf_comp = ndf.compute()
206207
assert list(ndf.columns) == list(ndf_comp.columns)
207208
assert list(ndf["nested"].nest.fields) == list(ndf["nested"].nest.fields)
209+
assert_frame_equal(ndf_comp.iloc[:0], ndf.meta)
208210

209211
# Check with just list_columns
210212
ndf = nd.NestedFrame.from_lists(nf, list_columns=["e", "f"])

0 commit comments

Comments
 (0)