Skip to content

Preparing to release bottleneck 1.2.1 #168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
kwgoodman opened this issue May 1, 2017 · 16 comments
Closed

Preparing to release bottleneck 1.2.1 #168

kwgoodman opened this issue May 1, 2017 · 16 comments

Comments

@kwgoodman
Copy link
Collaborator

kwgoodman commented May 1, 2017

I am getting ready to release bottleneck 1.2.1. This release adds support for NumPy's relaxed strides checking and fixes a few bugs. The only thing left to do is testing.

If you guys (@cgohlke windows, @shoyer xarray and pandas, @toobaz debian) have time please test.

Everyone else, please send in test reports. Does bn.test() pass on your computer? Does bn 1.2.1 (the master branch) run with your code that uses bn?

@cgohlke
Copy link
Contributor

cgohlke commented May 1, 2017

All bottleneck tests pass on Windows with numpy+MKL-1.11.3.

@itdaniher
Copy link

All tests pass on Linux / AArch64 / Numpy 1.11.0 on Python3.5.2.

@kwgoodman
Copy link
Collaborator Author

kwgoodman commented May 5, 2017

Thanks to @cgohlke I added another bug fix #163 to this release.

@itdaniher and @cgohlke could you test again? And @shoyer do you know someone who can test this release against xarray and pandas?

@itdaniher
Copy link

itdaniher commented May 5, 2017

169 (edit: all) tests passed on commit f01a64b / Linux / AArch64 / Numpy 1.11 / Python3.5.2.

@cgohlke
Copy link
Contributor

cgohlke commented May 5, 2017

All tests pass on Windows with numpy+mkl-1.11.3. This also fixes pandas-dev/pandas#15453.

@kwgoodman
Copy link
Collaborator Author

The recent bug fix #163 adds the L for long:

In [1]: bn.nanmin(1)
Out[1]: 1L
In [2]: bn.nansum(1)
Out[2]: 1L

Not sure what I think of that. I could make the bugfix windows only.

@cgohlke
Copy link
Contributor

cgohlke commented May 5, 2017

The recent bug fix #163 adds the L for long:

That is only on Python 2.7. It would be nice to return PyInt objects if possible. PyLong objects are needed on Windows and 32-bit Unix if the asum etc values are 64-bit and larger than LONG_MAX.

@kwgoodman
Copy link
Collaborator Author

OK, that's good to know. Well, the bug fix means that we are now at least returning the correct number. That's good enough for me for this release.

@toobaz
Copy link
Contributor

toobaz commented May 6, 2017

I can confirm a debian package of f01a64b on amd64 debian testing passes all tests.

@kwgoodman
Copy link
Collaborator Author

Thanks, guys, for all the testing.

I am surprised we haven't heard from @shoyer (he requested this release). I'd like to get a developer of pandas and xarray to test the release. Would be nice if one of them would watch this (low traffic) repo.

@shoyer
Copy link
Member

shoyer commented May 9, 2017

Sorry, I haven't had time to get to this. I did open up pandas and xarray issues, hopefully someone will step up.

@kwgoodman
Copy link
Collaborator Author

Yeah, I figured it was just that you were busy. Either that or your notifications were coming at you like a firehose. Or maybe both.

@shoyer
Copy link
Member

shoyer commented May 10, 2017

xarray passes with bottleneck 1.2.1. @fmaussion added a test against bottleneck master to our Travis-CI configuration, so this should be basically trivial to verify in the future!

@kwgoodman
Copy link
Collaborator Author

kwgoodman commented May 15, 2017

Bottleneck 1.2.1 is tagged and released. We'll go without pandas testing for this bug fix release.

Note: I only test (appveyor and travis) with py 2.7, 3.5, 3.6 and numpy 1.12.1. So that is what is officially supported. A bug in numpy 1.12.0 will cause a unit test failure in bottleneck.

Thanks to all who tested.

@toobaz
Copy link
Contributor

toobaz commented May 15, 2017

We'll go without pandas testing for this bug fix release.

I just ran the pandas test suite: all good.

@shoyer
Copy link
Member

shoyer commented May 15, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants