Skip to content

Update sizeof definitions#3582

Merged
mrocklin merged 1 commit intodask:masterfrom
mrocklin:update-sizeof
Jun 10, 2018
Merged

Update sizeof definitions#3582
mrocklin merged 1 commit intodask:masterfrom
mrocklin:update-sizeof

Conversation

@mrocklin
Copy link
Copy Markdown
Member

@mrocklin mrocklin commented Jun 8, 2018

Previously we had two independent implementations in dask/dask and
dask/distributed that had diverged. This brings them back in sync.

We'll remove the version in dask/distributed shortly to avoid this from
happening again.

  • Tests added / passed
  • Passes flake8 dask

Previously we had two independent implementations in dask/dask and
dask/distributed that had diverged.  This brings them back in sync.

We'll remove the version in dask/distributed shortly to avoid this from
happening again.
mrocklin added a commit to mrocklin/distributed that referenced this pull request Jun 8, 2018
Previously we copied these implementations between dask/dask and
dask/distributed.  Now we depend explicitly on the dask/dask
implementation.

Depends on dask/dask#3582
@mrocklin mrocklin merged commit daf6fae into dask:master Jun 10, 2018
@mrocklin mrocklin deleted the update-sizeof branch June 10, 2018 14:32
mrocklin added a commit to dask/distributed that referenced this pull request Jun 10, 2018
Previously we copied these implementations between dask/dask and
dask/distributed.  Now we depend explicitly on the dask/dask
implementation.

Depends on dask/dask#3582
convexset added a commit to convexset/dask that referenced this pull request Jun 12, 2018
* master:
  [array.linalg.qr] added sfqr (short-and-fat) as a counterpart to tsqr… (dask#3575)
  Drop Dask Array's `learn` module (dask#3580)
  Adds `isneginf` and `isposinf` (dask#3581)
  Update sizeof definitions (dask#3582)
  BUG: do not raise IndexError for identity slice in array.vindex (dask#3559)
  Correct internal dimension of Dask's SVD (dask#3517)
  Fix check in configuration with env=None (dask#3562)
  Dataframe sample method docstring fix (dask#3566)
  Include "region" in key names for da.store (dask#3540)

# Conflicts:
#	dask/array/linalg.py
#	dask/array/tests/test_linalg.py
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.

1 participant