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/791#issuecomment-203969194,https://api.github.com/repos/pydata/xarray/issues/791,203969194,MDEyOklzc3VlQ29tbWVudDIwMzk2OTE5NA==,4295853,2016-03-31T14:40:48Z,2016-03-31T14:40:48Z,CONTRIBUTOR,"Please see https://github.com/pydata/xarray/pull/812 for the PR stub. Note, I still need to work on this when I get some time. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928 https://github.com/pydata/xarray/issues/791#issuecomment-202513679,https://api.github.com/repos/pydata/xarray/issues/791,202513679,MDEyOklzc3VlQ29tbWVudDIwMjUxMzY3OQ==,4295853,2016-03-28T18:12:28Z,2016-03-28T18:12:28Z,CONTRIBUTOR,"Thanks @jhamman! ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928 https://github.com/pydata/xarray/issues/791#issuecomment-202507482,https://api.github.com/repos/pydata/xarray/issues/791,202507482,MDEyOklzc3VlQ29tbWVudDIwMjUwNzQ4Mg==,4295853,2016-03-28T17:55:37Z,2016-03-28T17:55:37Z,CONTRIBUTOR,"@shoyer and @jhamman, I've started a branch to address this at https://github.com/pwolfram/xarray/tree/add_cumsum_cumprod, but there appear to be some other issues that may need resolved prior to this implementation, e.g., https://github.com/pydata/xarray/issues/807. Also, 2 quick new developer questions: 1. Is there any easy way to do xarray development short of doing the ful xarray install via something like `pip install --upgrade git+ssh://git@github.com/pwolfram/numpy@nancumsumprod`? 2. How can I run the test suite (I'm thinking it may be something like `python runtests.py -v` for numpy)? I suspect there is a simple way to do these things but would appreciate hearing about your workflows. Thanks! ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928 https://github.com/pydata/xarray/issues/791#issuecomment-195611382,https://api.github.com/repos/pydata/xarray/issues/791,195611382,MDEyOklzc3VlQ29tbWVudDE5NTYxMTM4Mg==,4295853,2016-03-12T00:09:15Z,2016-03-12T00:09:15Z,CONTRIBUTOR,"@shoyer, should a custom version supporting nans be implemented too? I think that is more useful for my immediate needs and it would be good to have both options for completeness. However, it will be somewhat of a hack and involve adding `nancumsum` and `nancumprod` functions to `npcompat.py`. Note, however, that these are not standard numpy functions, even in known future versions, so I'm concerned this change falls into the ""hack"" category. This being said, implementation of the standard numpy functions would still be useful and the nans case could be handled at the application level. Can you provide some perspective on what you would be willing to admit into xarray? It seems like standard `cumsum` and `cumprod` are safe bets but the nan versions are somewhat suspect. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928 https://github.com/pydata/xarray/issues/791#issuecomment-195440644,https://api.github.com/repos/pydata/xarray/issues/791,195440644,MDEyOklzc3VlQ29tbWVudDE5NTQ0MDY0NA==,4295853,2016-03-11T16:33:11Z,2016-03-11T16:33:11Z,CONTRIBUTOR,"Note, numpy doesn't provide `cumsum` or `cumprod` that are `nan` compatible. I just checked https://github.com/numpy/numpy/blob/master/numpy/lib/nanfunctions.py#L4-L17 so injecting logic may be the way to go. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928 https://github.com/pydata/xarray/issues/791#issuecomment-195440103,https://api.github.com/repos/pydata/xarray/issues/791,195440103,MDEyOklzc3VlQ29tbWVudDE5NTQ0MDEwMw==,4295853,2016-03-11T16:31:05Z,2016-03-11T16:31:05Z,CONTRIBUTOR,"You are correct that it shouldn't be too difficult. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928 https://github.com/pydata/xarray/issues/791#issuecomment-195439946,https://api.github.com/repos/pydata/xarray/issues/791,195439946,MDEyOklzc3VlQ29tbWVudDE5NTQzOTk0Ng==,4295853,2016-03-11T16:30:28Z,2016-03-11T16:30:39Z,CONTRIBUTOR,"It may be more involved, e.g., there appears to be a custom `nanprod` called by https://github.com/pydata/xarray/blob/master/xarray/core/ops.py#L374-L383 at https://github.com/pydata/xarray/blob/master/xarray/core/npcompat.py#L186-L253. I could inject logic to remove the `nan` prepending for these functions at https://github.com/pydata/xarray/blob/master/xarray/core/ops.py#L310-L352 but this may be somewhat of a hack... ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928 https://github.com/pydata/xarray/issues/791#issuecomment-195419072,https://api.github.com/repos/pydata/xarray/issues/791,195419072,MDEyOklzc3VlQ29tbWVudDE5NTQxOTA3Mg==,4295853,2016-03-11T15:37:17Z,2016-03-11T15:37:17Z,CONTRIBUTOR,"Advice on the feasibility of adding cumsum / cumprod appreciated. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,140214928