-
Notifications
You must be signed in to change notification settings - Fork 15
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
Move Scippneutron's elem_unit
and elem_dtype
to Scipp?
#3391
Comments
I was actually wondering if |
Pretty sure the dtype shouldn't. Otherwise, code that checks dtypes also has to check for |
I can't think of many examples where we are doing a check for the exotic dtypes you get from binned data, can you? And for those that do, maybe a more dedicated method or property would be a viable alternative? |
My argument is the other way around. Using |
Isn't that what we want, in most situations? But I suppose the question is: If binned data is not described by the I think it is worth thinking about this again. While it is true that we make binned data part of the dtype, this is not necessarily done consistently, since we have found it more than inconvenient in several places (e.g., # da.sizes == {'x': 2, 'y':3}
da = da.bins.sum() but one could just as well write # da.sizes == {'x': 2, 'y':3, 'event':None}
da = da.sum('event') Now, it is unfortunately not as easy as that, since we almost always use a |
Can we move Scippneutron's
elem_unit
andelem_dtype
to Scipp?They are quite useful.
See also #3029
The text was updated successfully, but these errors were encountered: