-
Notifications
You must be signed in to change notification settings - Fork 59
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
BUG: Python stops working when importing utide on Windows platform #30
Comments
I narrowed it down to the |
I downgrading to @efiring |
Thanks, @ocefpaf ! That also worked for me. |
https://groups.google.com/a/continuum.io/forum/#!topic/anaconda/0OMcBlfWR_A might be the same thing. |
Scipy 0.17 failed to read ut_constants.mat on Windows platform. Please refer to the attachment to check the log file and the script. Traceback (most recent call last): |
@saulomeirelles, somehow you have gotten an html file in place of ut_constants.mat. The real ut_constants.mat is small:
The file in your zipfile, which gives me the "version 47,47" error when I try to read it with scipy 0.16, is large:
and by opening it in a text editor you can verify that it is html. |
Ough! My bad! I'll check that asap. Sent from my smartphone. Please forgive typos.
|
Hi @efiring, Python stopped working when loading ut_constant.mat. I did not even get the trace back message in the log file. You can find the script and other files in the attachment. I've tested in different Scipy version. Apparently, Python crashes in all versions > 0.16.0. |
@saulomeirelles Would you submit this as an issue to https://github.com/scipy/scipy/issues, please? Put the sample matfile in a gist (I think that is the right mechanism--@ocefpaf can advise) and link to that, as well as to the google groups message I noted in an earlier comment. |
@saulomeirelles If you have time for some more testing, you could check some simpler matfiles to see if you can narrow down the problem. Is it the presence of unicode? Is it the nested structure? In any case, you can create the issue first and then add comments based on additional testing later. This is an important regression in scipy, but not one that will be immediately recognized by most scipy developers--like me, most probably don't use Windows. |
Saulo, I am traveling (without a computer), but as, Eric said, that must be
|
@efiring Would you be OK to provide a |
I think there can be python2/3 compatibility problems with npz. If those problems are not triggered by what we need to store here, then using npz is fine. Otherwise we could switch to netcdf and use the scipy netcdf reader. I could look into this in the next couple of days; but if you want to do it, that's fine too. |
I have a PR for |
@saulomeirelles you can install the latest version using conda and the conda-forge community channel: conda install -c conda-forge utide That should work on Windows now. |
Hi everyone,
I recently updated my Utide package on Windows 7 64-bit. Now every time I import utide, Python simply stops working and the program closes down. @ocefpaf has done some tests and confirmed the bug. I am using the following versions:
Scipy 0.17.0
Python 2.7.11
UTide 0.1
I ran my scripts on Linux and all went well.
Cheers, Saulo
The text was updated successfully, but these errors were encountered: