catalystneuro / lazy_ops

Lazy transposing and slicing of h5py and Zarr Datasets
BSD 3-Clause "New" or "Revised" License
3 stars 3 forks source link

setting property to self #11

Closed bendichter closed 4 years ago

bendichter commented 4 years ago

This is an unusual thing to do:

https://github.com/ben-dichter-consulting/lazy_ops/blob/38432436dfa3eced8d6a2a73062f303bb05e154e/lazy_ops/lazy_loading.py#L38

Is there a reason for this? Why can't you just use self instead of self._lazy_slice?

d-sot commented 4 years ago

This was a leftover from a much earlier iteration. It should be fixed in 0ad986a364ace52ffa8ecd888106dea0ec581dbf