Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix representation of AsType's dtypes #120

Merged
merged 1 commit into from Feb 21, 2017
Merged

Fix representation of AsType's dtypes #120

merged 1 commit into from Feb 21, 2017

Conversation

jakirkham
Copy link
Member

Use string representations of AsType's dtypes instead of allowing the dtypes to be converted automatically. This matches more closely with what Delta is doing in its representation.

@jakirkham jakirkham mentioned this pull request Feb 21, 2017
Merged
@jakirkham
Copy link
Member Author

I know we are working on moving this to Numcodecs, but I'd still very much like to see this merged if that is ok.

@alimanfoo alimanfoo merged commit 1bce5da into zarr-developers:master Feb 21, 2017
@jakirkham jakirkham deleted the fix_astype_repr branch February 21, 2017 14:27
@jakirkham
Copy link
Member Author

Thanks @alimanfoo. Much appreciated.

@alimanfoo alimanfoo modified the milestone: v2.2 Feb 24, 2017
@alimanfoo alimanfoo mentioned this pull request Oct 24, 2017
Closed
4 tasks
@alimanfoo alimanfoo added enhancement New features or improvements release notes done Automatically applied to PRs which have release notes. labels Nov 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New features or improvements release notes done Automatically applied to PRs which have release notes.
Projects
None yet
Milestone
v2.2
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jakirkham @alimanfoo