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/636#issuecomment-153821045,https://api.github.com/repos/pydata/xarray/issues/636,153821045,MDEyOklzc3VlQ29tbWVudDE1MzgyMTA0NQ==,1217238,2015-11-04T18:28:16Z,2015-11-04T18:28:16Z,MEMBER,"Great news! Thank you Dave! On Wed, Nov 4, 2015 at 10:27 AM, Dave Brown notifications@github.com wrote: > The PyNIO license has been rewritten as a standard 3-clause BSD license. > Hopefully this satisfies all concerns. > > http://www.pyngl.ucar.edu/Licenses/PyNIO_source_license.shtml > > We will put it on the github page when it is made public. > > On Thu, Oct 29, 2015 at 2:40 PM, Stephan Hoyer notifications@github.com > wrote: > > > I agree that the license seems _complaint_ with OSI guidelines, but > > nonetheless it isn't an OSI approved license. These FAQ entries seem > > relevant: http://opensource.org/faq#approved-licenses-only > > > > — > > Reply to this email directly or view it on GitHub > > https://github.com/xray/xray/pull/636#issuecomment-152312224. > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-153820521. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-153820521,https://api.github.com/repos/pydata/xarray/issues/636,153820521,MDEyOklzc3VlQ29tbWVudDE1MzgyMDUyMQ==,2020717,2015-11-04T18:27:16Z,2015-11-04T18:27:16Z,NONE,"The PyNIO license has been rewritten as a standard 3-clause BSD license. Hopefully this satisfies all concerns. http://www.pyngl.ucar.edu/Licenses/PyNIO_source_license.shtml We will put it on the github page when it is made public. On Thu, Oct 29, 2015 at 2:40 PM, Stephan Hoyer notifications@github.com wrote: > I agree that the license seems _complaint_ with OSI guidelines, but > nonetheless it isn't an OSI approved license. These FAQ entries seem > relevant: http://opensource.org/faq#approved-licenses-only > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-152312224. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152320644,https://api.github.com/repos/pydata/xarray/issues/636,152320644,MDEyOklzc3VlQ29tbWVudDE1MjMyMDY0NA==,2020717,2015-10-29T20:55:01Z,2015-10-29T20:55:01Z,NONE,"OK I'll pass along that information. On Thu, Oct 29, 2015 at 2:40 PM, Stephan Hoyer notifications@github.com wrote: > I agree that the license seems _complaint_ with OSI guidelines, but > nonetheless it isn't an OSI approved license. These FAQ entries seem > relevant: http://opensource.org/faq#approved-licenses-only > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-152312224. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152312224,https://api.github.com/repos/pydata/xarray/issues/636,152312224,MDEyOklzc3VlQ29tbWVudDE1MjMxMjIyNA==,1217238,2015-10-29T20:40:11Z,2015-10-29T20:40:11Z,MEMBER,"I agree that the license seems _complaint_ with OSI guidelines, but nonetheless it isn't an OSI approved license. These FAQ entries seem relevant: http://opensource.org/faq#approved-licenses-only ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152308371,https://api.github.com/repos/pydata/xarray/issues/636,152308371,MDEyOklzc3VlQ29tbWVudDE1MjMwODM3MQ==,2020717,2015-10-29T20:26:14Z,2015-10-29T20:26:14Z,NONE,"I was just informed that ""our lawyers looked it over and said we're OSI compliant."" Specifically ""He compared the wording of both ours and the BSD one that you referenced, and said they are about 98% similar. The other 2%, which is something that's in the BSD that's not in ours, doesn't cause our license to be non-compliant."" Do you think this works? I do think we should fix the bad link to GRIBEX and point to the current ECMWF license. -dave On Thu, Oct 29, 2015 at 2:16 PM, David Brown dbrown@ucar.edu wrote: > We are looking into our options on this. I know there were NCAR legal > people involved in producing the current license. Hopefully given the > current environment we will be to put it into a standard form that clearly > meets the OSI requirement. > > On Thu, Oct 29, 2015 at 11:14 AM, Filipe notifications@github.com wrote: > > > but it would be nice to be able to say that you use a standard license > > > > [image: :+1:] > > > > — > > Reply to this email directly or view it on GitHub > > https://github.com/xray/xray/pull/636#issuecomment-152253168. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152306213,https://api.github.com/repos/pydata/xarray/issues/636,152306213,MDEyOklzc3VlQ29tbWVudDE1MjMwNjIxMw==,2020717,2015-10-29T20:16:59Z,2015-10-29T20:16:59Z,NONE,"We are looking into our options on this. I know there were NCAR legal people involved in producing the current license. Hopefully given the current environment we will be to put it into a standard form that clearly meets the OSI requirement. On Thu, Oct 29, 2015 at 11:14 AM, Filipe notifications@github.com wrote: > but it would be nice to be able to say that you use a standard license > > [image: :+1:] > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-152253168. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152253168,https://api.github.com/repos/pydata/xarray/issues/636,152253168,MDEyOklzc3VlQ29tbWVudDE1MjI1MzE2OA==,950575,2015-10-29T17:14:08Z,2015-10-29T17:14:08Z,CONTRIBUTOR,"> but it would be nice to be able to say that you use a standard license :+1: ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152251450,https://api.github.com/repos/pydata/xarray/issues/636,152251450,MDEyOklzc3VlQ29tbWVudDE1MjI1MTQ1MA==,1217238,2015-10-29T17:08:32Z,2015-10-29T17:08:32Z,MEMBER,"Actually I looked a little more carefully and the PyNIO license is not quite [BSD](https://en.wikipedia.org/wiki/BSD_licenses) -- the disclaimer of liability at the bottom is different. It looks like it's actually taken from the [MIT license](https://en.wikipedia.org/wiki/MIT_License), with a few words copied from BSD: ""INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES"". In any case, both MIT and BSD are OSI approved with similar restrictions, so I'm sure this license is also fine. I have not idea if this license is original to PyNIO or not, but it would be nice to be able to say that you use a standard license -- it makes this stuff a bit simpler. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152240832,https://api.github.com/repos/pydata/xarray/issues/636,152240832,MDEyOklzc3VlQ29tbWVudDE1MjI0MDgzMg==,1217238,2015-10-29T16:37:53Z,2015-10-29T16:59:49Z,MEMBER,"Aside from the notice at the top ""PLEASE READ THIS SOFTWARE LICENSE CAREFULLY"" the license is ~~direct~~ _almost exact_ copy of 3 clause BSD. It would be great to get this in the readme (once you're on github) to make this entirely clear. The link for the license to GRIBX no longer works on the ECMWF website. But some googling turns up source files with the Apache 2.0 license attached, so I think we're in the clear (Apache 2.0 is BSD compatible). In any case, it should be in the PyNIO source code. For easier reference, it would be better to copy in the text of any referenced licenses in the PyNIO license itself, though. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152242230,https://api.github.com/repos/pydata/xarray/issues/636,152242230,MDEyOklzc3VlQ29tbWVudDE1MjI0MjIzMA==,2020717,2015-10-29T16:40:57Z,2015-10-29T16:40:57Z,NONE,"OK thanks for the info and the doc suggestions. On Thu, Oct 29, 2015 at 10:37 AM, Stephan Hoyer notifications@github.com wrote: > Aside from the notice at the top ""PLEASE READ THIS SOFTWARE LICENSE > CAREFULLY"" the license is direct copy of 3 clause BSD. It would be great to > get this in the readme (once you're on github) to make this entirely clear. > > The link for the license to GRIBX no longer works on the ECMWF website. > But some googling turns up source files with the Apache 2.0 license > attached, so I think we're in the clear (Apache 2.0 is BSD compatible). In > any case, it should be in the PyNIO source code. For easier reference, it > would be better to copy in the text of any referenced licenses in the PyNIO > license itself, though. > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-152240832. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152228485,https://api.github.com/repos/pydata/xarray/issues/636,152228485,MDEyOklzc3VlQ29tbWVudDE1MjIyODQ4NQ==,2020717,2015-10-29T16:08:49Z,2015-10-29T16:08:49Z,NONE,"Good question. I don't know. I do know it was intended to be as open as BSD, but I will need to research further what the actual situation is. On Thu, Oct 29, 2015 at 10:03 AM, Filipe notifications@github.com wrote: > The license here: > http://www.pyngl.ucar.edu/Licenses/PyNIO_source_license.shtml > > @david-ian-brown https://github.com/david-ian-brown sorry for my > ignorance but is that OSI approved http://opensource.org/licenses? > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-152226596. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152226596,https://api.github.com/repos/pydata/xarray/issues/636,152226596,MDEyOklzc3VlQ29tbWVudDE1MjIyNjU5Ng==,950575,2015-10-29T16:03:22Z,2015-10-29T16:03:22Z,CONTRIBUTOR,"> The license here: http://www.pyngl.ucar.edu/Licenses/PyNIO_source_license.shtml @david-ian-brown sorry for my ignorance but is that [OSI approved](http://opensource.org/licenses)? ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152225972,https://api.github.com/repos/pydata/xarray/issues/636,152225972,MDEyOklzc3VlQ29tbWVudDE1MjIyNTk3Mg==,2020717,2015-10-29T16:00:59Z,2015-10-29T16:00:59Z,NONE,"The license here: http://www.pyngl.ucar.edu/Licenses/PyNIO_source_license.shtml On Wed, Oct 28, 2015 at 3:51 PM, Stephan Hoyer notifications@github.com wrote: > @david-ian-brown https://github.com/david-ian-brown What license are > you using for PyNIO? It looks like > https://www.pyngl.ucar.edu/Licenses/PyNIO_source_license.shtml a > standard 3 clause BSD? This would be helpful information to highlight on > the new GitHub site -- I could only finding it by searching google, not by > navigating on the website. > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-152005733. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152005733,https://api.github.com/repos/pydata/xarray/issues/636,152005733,MDEyOklzc3VlQ29tbWVudDE1MjAwNTczMw==,1217238,2015-10-28T21:51:49Z,2015-10-28T21:51:49Z,MEMBER,"@david-ian-brown What license are you using for PyNIO? It [looks like](https://www.pyngl.ucar.edu/Licenses/PyNIO_source_license.shtml) a standard 3 clause BSD? This would be helpful information to highlight on the new GitHub site -- I could only finding it by searching google, not by navigating on the website. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152002550,https://api.github.com/repos/pydata/xarray/issues/636,152002550,MDEyOklzc3VlQ29tbWVudDE1MjAwMjU1MA==,950575,2015-10-28T21:40:54Z,2015-10-28T21:40:54Z,CONTRIBUTOR,"Thanks @david-ian-brown! That is good news!! ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-152001573,https://api.github.com/repos/pydata/xarray/issues/636,152001573,MDEyOklzc3VlQ29tbWVudDE1MjAwMTU3Mw==,2020717,2015-10-28T21:38:31Z,2015-10-28T21:38:31Z,NONE,"Hi Filipe, We will be making the pynio github site public soon, and I will update the PyPI information when that happens. On Wed, Oct 28, 2015 at 3:51 AM, Filipe notifications@github.com wrote: > Maybe we could get ioos to also keep a pynio conda build up to date (cc > @ocefpaf https://github.com/ocefpaf) > > @david-ian-brown https://github.com/david-ian-brown is pynio open > source now? If so I can add it to the ioos channel, but at PyPI it is still > missing the source. > > — > Reply to this email directly or view it on GitHub > https://github.com/xray/xray/pull/636#issuecomment-151783551. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-151783551,https://api.github.com/repos/pydata/xarray/issues/636,151783551,MDEyOklzc3VlQ29tbWVudDE1MTc4MzU1MQ==,950575,2015-10-28T09:51:47Z,2015-10-28T09:51:47Z,CONTRIBUTOR,"> Maybe we could get ioos to also keep a pynio conda build up to date (cc @ocefpaf) @david-ian-brown is pynio open source now? If so I can add it to the ioos channel, but at PyPI it is still missing the source. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-151733579,https://api.github.com/repos/pydata/xarray/issues/636,151733579,MDEyOklzc3VlQ29tbWVudDE1MTczMzU3OQ==,1217238,2015-10-28T05:54:40Z,2015-10-28T05:54:40Z,MEMBER,"Thanks again @david-ian-brown for your help with this. We've already been finding being able to use xray with pynio to be quite useful -- being able to read grib files is a killer feature. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-151727064,https://api.github.com/repos/pydata/xarray/issues/636,151727064,MDEyOklzc3VlQ29tbWVudDE1MTcyNzA2NA==,2443309,2015-10-28T05:13:33Z,2015-10-28T05:13:33Z,MEMBER,"@shoyer - This looks good to me. @david-ian-brown - updating the conda channel shouldn't be a problem when/if that comes to be. Maybe we could get ioos to also keep a pynio conda build up to date (cc @ocefpaf) ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-151717704,https://api.github.com/repos/pydata/xarray/issues/636,151717704,MDEyOklzc3VlQ29tbWVudDE1MTcxNzcwNA==,2020717,2015-10-28T04:20:53Z,2015-10-28T04:20:53Z,NONE,"Great! Just FYI, the conda channel name will almost certainly change soon to something more representative of NCAR. I will let you know when that happens. Thanks for working with us on this. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-151688945,https://api.github.com/repos/pydata/xarray/issues/636,151688945,MDEyOklzc3VlQ29tbWVudDE1MTY4ODk0NQ==,1217238,2015-10-28T01:10:55Z,2015-10-28T01:10:55Z,MEMBER,"Good call. I added a small section to Serialization & IO: ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-151685286,https://api.github.com/repos/pydata/xarray/issues/636,151685286,MDEyOklzc3VlQ29tbWVudDE1MTY4NTI4Ng==,2443309,2015-10-28T00:45:13Z,2015-10-28T00:45:13Z,MEMBER,"I think this may benefit from some documentation, both in [Serialization and IO](http://xray.readthedocs.org/en/latest/io.html?highlight=h5netcdf#serialization-and-io) and in `open_dataset`. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-151681205,https://api.github.com/repos/pydata/xarray/issues/636,151681205,MDEyOklzc3VlQ29tbWVudDE1MTY4MTIwNQ==,1217238,2015-10-28T00:14:44Z,2015-10-28T00:14:44Z,MEMBER,"OK, great -- the units tests are now passing on travis, using numpy 1.9. There's no particularly good reason why we aren't testing against numpy 1.10 -- it just happens that I arbitrarily decided to test cdat-lite with the same build, and it only supports numpy 1.9 or older. If there are no objections I'll be merging this shortly. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-150668435,https://api.github.com/repos/pydata/xarray/issues/636,150668435,MDEyOklzc3VlQ29tbWVudDE1MDY2ODQzNQ==,2020717,2015-10-23T19:21:17Z,2015-10-23T19:21:17Z,NONE,"Hi Stephan, So in this message: RuntimeError: module compiled against API version a but this version of numpy is 9 the ""a"" actually means 10 in hex I suppose. I see that I need to do more thorough testing. My latest builds have all pulled in 1.10.x by default. I did put some NumPy specific versions up on the conda site before 1.10 was released. Maybe I need to make all the builds I do numpy (and python) version specific. I'll get a new 1.9 version up today. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315 https://github.com/pydata/xarray/pull/636#issuecomment-150119824,https://api.github.com/repos/pydata/xarray/issues/636,150119824,MDEyOklzc3VlQ29tbWVudDE1MDExOTgyNA==,1217238,2015-10-22T06:20:33Z,2015-10-22T06:20:33Z,MEMBER,"@david-ian-brown This is working for me on OS X, but I'm getting an error when I attempt to import your conda build of pynio on Linux: ``` >>> import Nio RuntimeError: module compiled against API version a but this version of numpy is 9 Traceback (most recent call last): File """", line 1, in File ""/export/data/envs/test_env/lib/python2.7/site-packages/PyNIO/Nio.py"", line 83, in from nio import * ImportError: numpy.core.multiarray failed to import ``` Can you reproduce this? I think this is an ABI compatibility issue (ugh!) related to building against one version of numpy but installing another. To that end, it's probably worth trying specifying specific numpy requirements. There were a few other issues that came up here that I'll note in specific comments on the code. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112677315