New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix len() error on dask array with ConstantArray of size 1. #373

Merged
merged 3 commits into from Aug 21, 2017

Conversation

Projects
None yet
1 participant
@rainwoodman
Member

rainwoodman commented Aug 21, 2017

This can be triggered when we generate an empty FOF catalog, and saving it to disk. The Selection column is a ConstantArray of size 1. But the squeeze method makes it a scalar -- which has no well defined len in dask. This PR removes the squeeze

@rainwoodman rainwoodman merged commit 03bc45c into bccp:master Aug 21, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 95.103%
Details

@rainwoodman rainwoodman deleted the rainwoodman:constarraysize branch Aug 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment