Skip to content

Commit

Permalink
Merge pull request #397 from martindurant/fixes
Browse files Browse the repository at this point in the history
Fix tests following change to LazyReferenceMapper
  • Loading branch information
martindurant authored Nov 17, 2023
2 parents 3c4e9fc + eeea5ba commit 498385c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
5 changes: 4 additions & 1 deletion kerchunk/df.py
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,10 @@ def refs_to_dataframe(

fs, _ = fsspec.core.url_to_fs(url, **(storage_options or {}))
out = LazyReferenceMapper.create(
record_size, root=url, fs=fs, categorical_threshold=categorical_threshold
record_size=record_size,
root=url,
fs=fs,
categorical_threshold=categorical_threshold,
)

for k in sorted(refs):
Expand Down
2 changes: 1 addition & 1 deletion kerchunk/tests/test_combine.py
Original file line number Diff line number Diff line change
Expand Up @@ -310,7 +310,7 @@ def test_lazy_filler(tmpdir, refs):

fs = fsspec.filesystem("file")
tmpdir = str(tmpdir)
out = LazyReferenceMapper.create(10, tmpdir, fs)
out = LazyReferenceMapper.create(record_size=10, root=tmpdir, fs=fs)

mzz = MultiZarrToZarr(
[refs["single1"], refs["single2"]],
Expand Down

0 comments on commit 498385c

Please sign in to comment.