Skip to content

Added negative slicing capabilities to _slice_1d().#1

Merged
mrocklin merged 1 commit intomrocklin:slicingfrom
nevermindewe:master
Jan 22, 2015
Merged

Added negative slicing capabilities to _slice_1d().#1
mrocklin merged 1 commit intomrocklin:slicingfrom
nevermindewe:master

Conversation

@nevermindewe
Copy link
Contributor

This should complete dask block slicing.
Many test cases have been added.
Python 2 does not have an accumulate() in itertools, but python 3 does,
so I added the reference implementation to compatibility.py

This should complete dask block slicing.
Many test cases have been added.
Python 2 does not have an accumulate() in itertools, but python 3 does,
  so I added the reference implementation to compatibility.py
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool. I actually had to verify these using an interpreter. Good to know that we're beyond what's intuitive.

@mrocklin
Copy link
Owner

This looks great

mrocklin added a commit that referenced this pull request Jan 22, 2015
Added negative slicing capabilities to _slice_1d().
@mrocklin mrocklin merged commit 5a0f92f into mrocklin:slicing Jan 22, 2015
mrocklin pushed a commit that referenced this pull request Feb 21, 2015
mrocklin pushed a commit that referenced this pull request Jun 30, 2015
mrocklin pushed a commit that referenced this pull request Apr 5, 2017
Add index=False description to read_parquet
mrocklin pushed a commit that referenced this pull request Apr 18, 2019
* Add name kwarg to from_zarr (#1)

* Add name kwarg to from_zarr

* added default from_zarr naming convention per jrbourbeau

* flake8 fix

* Added test for from_zarr name hashing

* added name test

* update from_zarr docstring

* grammer curl

Co-Authored-By: mpeaton <mpeaton@users.noreply.github.com>

* python-snappy
mrocklin pushed a commit that referenced this pull request Jul 21, 2025
Updates `ReadParquet` to use metadata-parsing and IO logic from `dask.dataframe.io.parquet`.

Requires dask#9637 (only because my environment was using that PR when I put this together).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants