uxarray type dropped during `xr.Dataset + UxDataArray` and `ufunc(xr.DataArray, UxDataset)`
Maintainer thường phản hồi trong vòng 5 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Run the provided MCVE for the three mixed xarray/uxarray cases and inspect xarray method dispatch together with NumPy array_ufunc handling. Determine the xarray hook needed for these operations to preserve UxDataset, then verify that all listed binary operations and ufunc examples return UxDataset.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Version
v2026.09.1
How did you install UXarray?
Source
What happened?
Spinoff of #1685 and #1695 to track a few remaining difficult-to-solve cases, which should now be considered out of scope for those issues. Results from binary operations with one xarray object and one uxarray object, when one is a Dataset and the other is a DataArray, currently do not always return a UxDataset (but they should).
(Workaround for any users encountering this bug: explicitly cast objects to uxarray types before or after performing the operation.)
These cases are:
xr.Dataset * UxDataArray, where * can be any binary operation (e.g. multiplication, addition, subtraction)ufunc(xr.Dataset, UxDataArray), where ufunc can be any numpy binary ufunc (e.g.np.minimum,np.divmod)ufunc(xr.DataArray, UxDataset), where ufunc can be any numpy binary ufunc (e.g.np.minimum,np.divmod)
In the docstring of test_nondestructive_binary_ops_and_binary_ufuncs_output_types() in #1767, these cases map to:
- case (K)
- also case (K)
- case (J)
All of these issues cannot be fixed in uxarray directly. Rather, to get these properly fixed would require editing xarray directly to, at minimum, provide a hook which subclasses can use in these cases.
(1) cannot be fixed in uxarray directly because the operation goes directly to xr.Dataset's methods, and when xr.Dataset sees that the second term is an instance of xr.DataArray it decides to handle everything, without giving the second term a chance to alter the behavior in any way.
(2) and (3) cannot be fixed in uxarray directly because numpy's __array_ufunc__ sees that the first term is not a subclass of the second term (or vice versa), so it starts trying to apply __array_ufunc__ in order of inputs. This leads to applying the xarray object's __array_ufunc__, which similarly decides to handle everything after seeing that the second term is an instance of an xarray object (xr.DataArray in (2); xr.Dataset in (3)), without giving the second term a chance to alter the behavior in any way.
What did you expect to happen?
I expected these cases to all return objects with type UxDataset.
Can you provide a MCVE to repoduce the bug?
import numpy as np
import uxarray as ux
uxds = ux.tutorial.open_dataset('quad-hexagon')
uxda = uxds['t2m']
xrds = uxds.to_xarray()
xrda = uxda.to_xarray()
# case 1
print(type(xrds * uxda)) # expected UxDataset, got xr.Dataset
# case 2
print(type(np.minimum(xrds, uxda))) # expected UxDataset, got xr.Dataset
# case 3
print(type(np.maximum(xrda, uxds))) # expected UxDataset, got xr.Dataset
- Ngôn ngữ chính
- Python
- Star
- 229
- Fork
- 56
- Merge trung bình
- 8 ngày 3 giờ
- Pull request đã merge (30 ngày)
- 15
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của UXARRAY/uxarray
-
Update installation docs to include new conda recipesCó thể đã có người làm @Sevans711 đã nhận 7 ngày trước. Đang mởdocumentation
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 82/100
UXARRAY/uxarray#1793 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 5 ngày
-
developer experience
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
Maintainer thường phản hồi trong vòng 5 ngày
-
zonal_mean() and weighted_mean() should return the same type of objectCó thể đã có người làm @Sevans711 đã nhận 16 ngày trước. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
UXARRAY/uxarray#1733 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 5 ngày
-
documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 5 ngày
-
API Reference docs page: How about using numbered headingsCó thể đã có người làm @erogluorhan đã nhận hôm nay. Đang mởdocumentation improvement
UXARRAY/uxarray#1815 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 5 ngày
Tất cả issue của UXARRAY/uxarray
Issue tương tự
-
Maven path-index: "Ambiguous or noncanonical artifact path" error does not report the offending pathĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
pulp/pulp_maven#524 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100
infinispan/langchain-infinispan#34 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
521xueweihan/HelloGitHub#3891 ·