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/6135#issuecomment-1037063748,https://api.github.com/repos/pydata/xarray/issues/6135,1037063748,IC_kwDOAMm_X8490FZE,12237157,2022-02-12T08:28:47Z,2022-02-12T08:28:47Z,CONTRIBUTOR,"> Nice work @aaronspring! I think this is ready to go. Thank you so much @spencerkclark for your guidance ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1093466537 https://github.com/pydata/xarray/pull/6135#issuecomment-1032848596,https://api.github.com/repos/pydata/xarray/issues/6135,1032848596,IC_kwDOAMm_X849kATU,12237157,2022-02-08T17:03:18Z,2022-02-08T17:03:18Z,CONTRIBUTOR,thank you very much for the guidance here. I hope this is now close to how you envisioned this. I merely wanted to get a small improvement in functionality and didnt expect to learn so much. thank you so far @spencerkclark ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1093466537 https://github.com/pydata/xarray/pull/6135#issuecomment-1025184642,https://api.github.com/repos/pydata/xarray/issues/6135,1025184642,IC_kwDOAMm_X849GxOC,12237157,2022-01-30T16:58:23Z,2022-02-01T15:54:29Z,CONTRIBUTOR,failing tests are unrelated to `cftime` IMO,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1093466537 https://github.com/pydata/xarray/pull/6135#issuecomment-1025163440,https://api.github.com/repos/pydata/xarray/issues/6135,1025163440,IC_kwDOAMm_X849GsCw,12237157,2022-01-30T15:09:52Z,2022-01-30T15:09:52Z,CONTRIBUTOR,pre-commit.ci autofix,"{""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 1, ""eyes"": 0}",,1093466537 https://github.com/pydata/xarray/pull/6135#issuecomment-1008437041,https://api.github.com/repos/pydata/xarray/issues/6135,1008437041,IC_kwDOAMm_X848G4cx,12237157,2022-01-09T22:33:03Z,2022-01-09T22:33:03Z,CONTRIBUTOR,thanks for the reviews so far @spencerkclark @Illviljan @max-sixty - I think I implemented the suggestions and am looking forward to your next reviews,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1093466537 https://github.com/pydata/xarray/pull/6135#issuecomment-1008285229,https://api.github.com/repos/pydata/xarray/issues/6135,1008285229,IC_kwDOAMm_X848GTYt,14371165,2022-01-09T12:07:05Z,2022-01-09T12:07:05Z,MEMBER,"Since the function isn't typed currently mypy has no clue what `n` and `freq` is supposed to be, so it just skips statically checking the method right now. And since no type checking has been done in the past those isinstance check have probably been added to makes sure the type is correct, but this is checked at runtime so it costs time. At runtime it usually is best to assume all inputs are typed correctly and avoid these instance checks. I suppose if the IDE supports type checking it could feel runtime-y but it probably still is just statically checks if the code changes you did are valid.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1093466537 https://github.com/pydata/xarray/pull/6135#issuecomment-1008277351,https://api.github.com/repos/pydata/xarray/issues/6135,1008277351,IC_kwDOAMm_X848GRdn,12237157,2022-01-09T11:13:01Z,2022-01-09T11:13:01Z,CONTRIBUTOR,"Would you remove the isinstance checks? I though that mypy does check at runtime, so it would detect a wrong input by the user. (I am still new to typing etc)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1093466537