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/issues/1624#issuecomment-1101015179,https://api.github.com/repos/pydata/xarray/issues/1624,1101015179,IC_kwDOAMm_X85BoCiL,26384082,2022-04-18T01:43:46Z,2022-04-18T01:43:46Z,NONE,"In order to maintain a list of currently relevant issues, we mark issues as stale after a period of inactivity

If this issue remains relevant, please comment here or remove the `stale` label; otherwise it will be marked as closed automatically
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264509098
https://github.com/pydata/xarray/issues/1624#issuecomment-652804739,https://api.github.com/repos/pydata/xarray/issues/1624,652804739,MDEyOklzc3VlQ29tbWVudDY1MjgwNDczOQ==,1217238,2020-07-02T06:10:40Z,2020-07-02T06:10:40Z,MEMBER,"The validation was implemented in #2261, but the docstring could still use improvement.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264509098
https://github.com/pydata/xarray/issues/1624#issuecomment-652764200,https://api.github.com/repos/pydata/xarray/issues/1624,652764200,MDEyOklzc3VlQ29tbWVudDY1Mjc2NDIwMA==,26384082,2020-07-02T03:44:27Z,2020-07-02T03:44:27Z,NONE,"In order to maintain a list of currently relevant issues, we mark issues as stale after a period of inactivity

If this issue remains relevant, please comment here or remove the `stale` label; otherwise it will be marked as closed automatically
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264509098
https://github.com/pydata/xarray/issues/1624#issuecomment-409774622,https://api.github.com/repos/pydata/xarray/issues/1624,409774622,MDEyOklzc3VlQ29tbWVudDQwOTc3NDYyMg==,3606852,2018-08-02T01:20:43Z,2018-08-02T01:20:43Z,NONE,"@shoyer Nice! Thanks for pointing it out, hadn't seen that PR.

I like how the checks have been abstracted with validator functions, seems much more flexible overall.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264509098
https://github.com/pydata/xarray/issues/1624#issuecomment-409756878,https://api.github.com/repos/pydata/xarray/issues/1624,409756878,MDEyOklzc3VlQ29tbWVudDQwOTc1Njg3OA==,1217238,2018-08-01T23:23:18Z,2018-08-01T23:23:35Z,MEMBER,"@rmsare thanks for taking a crack at this!

I added some related validations as part of this larger PR, but it's still a work in progress: https://github.com/pydata/xarray/pull/2261","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264509098
https://github.com/pydata/xarray/issues/1624#issuecomment-409755419,https://api.github.com/repos/pydata/xarray/issues/1624,409755419,MDEyOklzc3VlQ29tbWVudDQwOTc1NTQxOQ==,3606852,2018-08-01T23:14:18Z,2018-08-01T23:14:18Z,NONE,"This issue is pretty stale, but I've taken a crack at this. Not sure I like my approach. This doesn't seem like a high priority issue, but if we can triage my implementation with some feedback, happy to work it up into a PR.

[branch diff](https://github.com/pydata/xarray/compare/master...rmsare:check-arithmetic-join)

The idea was to add `_check_values` method to the options class that check the current values in `OPTIONS` against a dictionary which contains lists of accepted values. If there's no entry in that dictionary, any value is assumed valid. But values of incorrect type can be caught by a `_check_types` method, which checks for type consistency with the default values.

Both are called at the end of initialization to catch bad values before some function tries to use them in the new context.

Does xarray have any established value checking practices? Skimming through the code I couldn't find any.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264509098
https://github.com/pydata/xarray/issues/1624#issuecomment-335849507,https://api.github.com/repos/pydata/xarray/issues/1624,335849507,MDEyOklzc3VlQ29tbWVudDMzNTg0OTUwNw==,1217238,2017-10-11T15:27:56Z,2017-10-11T15:27:56Z,MEMBER,"> By scanning the code I gathered that the valid values (and their meanings) should be the same as align(join=...), but I'd like confirmation on that...

That's correct. We should update the documentation and add a check.

Using enums might have been a better idea here than string values, but that's somewhat rare in scipy/pandas code.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,264509098