html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue https://github.com/pydata/xarray/pull/7444#issuecomment-1460450199,https://api.github.com/repos/pydata/xarray/issues/7444,1460450199,IC_kwDOAMm_X85XDLOX,14808389,2023-03-08T16:22:19Z,2023-03-08T16:23:36Z,MEMBER,"I agree, those do indeed seem unrelated, and I have no idea why the python 3.11 macos CI takes that long to run.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1535387692 https://github.com/pydata/xarray/pull/7444#issuecomment-1460423854,https://api.github.com/repos/pydata/xarray/issues/7444,1460423854,IC_kwDOAMm_X85XDEyu,6628425,2023-03-08T16:07:39Z,2023-03-08T16:07:39Z,MEMBER,Thanks @keewis -- I pushed a fix. The documentation build succeeded. The other build failures look unrelated.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1535387692 https://github.com/pydata/xarray/pull/7444#issuecomment-1460178545,https://api.github.com/repos/pydata/xarray/issues/7444,1460178545,IC_kwDOAMm_X85XCI5x,14808389,2023-03-08T13:46:19Z,2023-03-08T13:46:19Z,MEMBER,"the docs failure is real, I think we need to update https://github.com/pydata/xarray/blob/821dc24b5f3ed91b843a634bf8513a26046269ef/doc/user-guide/weather-climate.rst#L236 to use the new parameter (not sure how exactly, though)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1535387692 https://github.com/pydata/xarray/pull/7444#issuecomment-1459195989,https://api.github.com/repos/pydata/xarray/issues/7444,1459195989,IC_kwDOAMm_X85W-ZBV,6628425,2023-03-08T02:34:37Z,2023-03-08T02:34:37Z,MEMBER,@keewis and @dcherian indeed I just wanted to give it another once over. I think it should be ready for review now. Thanks for your patience on this!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1535387692 https://github.com/pydata/xarray/pull/7444#issuecomment-1458219179,https://api.github.com/repos/pydata/xarray/issues/7444,1458219179,IC_kwDOAMm_X85W6qir,2448579,2023-03-07T13:57:20Z,2023-03-07T13:57:20Z,MEMBER,"Shall we merge? AFAICT all groupby and resample tests now pass on `upstream-dev`: ``` =========================== short test summary info ============================ FAILED xarray/tests/test_backends.py::TestNetCDF4Data::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestNetCDF4ViaDaskData::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestZarrDictStore::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestZarrDirectoryStore::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestZarrKVStoreV3::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestZarrDirectoryStoreV3::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestZarrDirectoryStoreV3::test_write_read_select_write - KeyError: 'var1' FAILED xarray/tests/test_backends.py::TestZarrDirectoryStoreV3FromPath::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestZarrDirectoryStoreV3FromPath::test_write_read_select_write - KeyError: 'var1' FAILED xarray/tests/test_backends.py::TestScipyInMemoryData::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestScipyFileObject::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestScipyFilePath::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestNetCDF3ViaNetCDF4Data::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestNetCDF4ClassicViaNetCDF4Data::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestGenericNetCDFData::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestH5NetCDFData::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestH5NetCDFFileObject::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_backends.py::TestH5NetCDFViaDaskData::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[360_day] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[365_day] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[366_day] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[all_leap] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[gregorian] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[julian] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[noleap] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[proleptic_gregorian] - Failed: DID NOT RAISE FAILED xarray/tests/test_cftimeindex.py::test_to_datetimeindex_out_of_range[standard] - Failed: DID NOT RAISE FAILED xarray/tests/test_coding_times.py::test_should_cftime_be_used_source_outside_range - Failed: DID NOT RAISE FAILED xarray/tests/test_conventions.py::TestCFEncodedDataStore::test_roundtrip_cftime_datetime_data - AssertionError: assert 'days since 1-01-01' == 'days since 0001-01-01' - days since 0001-01-01 ? --- + days since 1-01-01 FAILED xarray/tests/test_dataarray.py::TestDataArray::test_sel_float - NotImplementedError: float16 indexes are not supported FAILED xarray/tests/test_dataarray.py::TestDataArray::test_to_and_from_cdms2_sgrid - ValueError: operands could not be broadcast together with shapes (3,) (4,) FAILED xarray/tests/test_units.py::TestVariable::test_aggregation[int64-method_max] - TypeError: no implementation found for 'numpy.max' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestVariable::test_aggregation[int64-method_min] - TypeError: no implementation found for 'numpy.min' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_aggregation[float64-function_max] - TypeError: no implementation found for 'numpy.max' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_aggregation[float64-function_min] - TypeError: no implementation found for 'numpy.min' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_aggregation[int64-function_max] - TypeError: no implementation found for 'numpy.max' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_aggregation[int64-function_min] - TypeError: no implementation found for 'numpy.min' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_aggregation[int64-method_max] - TypeError: no implementation found for 'numpy.max' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_aggregation[int64-method_min] - TypeError: no implementation found for 'numpy.min' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_unary_operations[float64-round] - TypeError: no implementation found for 'numpy.round' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataArray::test_unary_operations[int64-round] - TypeError: no implementation found for 'numpy.round' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataset::test_aggregation[int64-method_max] - TypeError: no implementation found for 'numpy.max' on types that implement __array_function__: [] FAILED xarray/tests/test_units.py::TestDataset::test_aggregation[int64-method_min] - TypeError: no implementation found for 'numpy.min' on types that implement __array_function__: [] = 43 failed, 15089 passed, 1223 skipped, 168 xfailed, 74 xpassed, 473 warnings in 671.46s (0:11:11) = ```","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1535387692 https://github.com/pydata/xarray/pull/7444#issuecomment-1458210029,https://api.github.com/repos/pydata/xarray/issues/7444,1458210029,IC_kwDOAMm_X85W6oTt,2448579,2023-03-07T13:51:49Z,2023-03-07T13:51:49Z,MEMBER,"> TimeResampleGrouper seems to do something different from the one in https://github.com/pydata/xarray/pull/7561, The version in that PR is just a container for the factorize method. I'm waiting for this to get merged before continuing","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1535387692 https://github.com/pydata/xarray/pull/7444#issuecomment-1457979204,https://api.github.com/repos/pydata/xarray/issues/7444,1457979204,IC_kwDOAMm_X85W5v9E,14808389,2023-03-07T11:08:07Z,2023-03-07T11:08:07Z,MEMBER,"both the failing upstream-dev CI *and* the failing docs CI seem to be unrelated to this. The `TimeResampleGrouper` seems to do something different from the one in #7561, but I think that's up to that PR to figure out (since we don't yet expose it as public API). So considering that, how much do you think is needed until we can merge this? And can I help with anything?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1535387692