-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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: add endfunction, endsubroutine to valid fortran end words #15035
Conversation
@@ -542,7 +542,7 @@ | |||
'ARRSIZE'] = '#define ARRSIZE(dims,rank) (_PyArray_multiply_list(dims,rank))' | |||
cppmacros['OLDPYNUM'] = """\ | |||
#ifdef OLDPYNUM | |||
#error You need to install NumPy version 13 or higher. See https://scipy.org/install.html | |||
#error You need to install NumPy version 0.13 or higher. See https://scipy.org/install.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
def test_module(self): | ||
k = np.array([1, 2, 3], dtype = np.float) | ||
w = np.array([1, 2, 3], dtype = np.float) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not make #6103 worse ;)
w = np.array([1, 2, 3], dtype = np.float) | |
w = np.array([1, 2, 3], dtype=float) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
heh, fixing
k = np.array([1, 2, 3], dtype = np.float) | ||
w = np.array([1, 2, 3], dtype = np.float) | ||
k = np.array([1, 2, 3], dtype = np.float64) | ||
w = np.array([1, 2, 3], dtype = np.float64) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PEP8 says no spaces around =
here, as in my suggested change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not fortran expert so can't sign off on the validity of the fortran syntax we're parsing, but the changes to the python look fine.
@melissawm maybe you can give a hand at reviewing this? |
Sure, just give me a few days since I'm a bit busy at work right now, if that's ok! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Thanks everyone for the review! |
Test, fix for missing end words in fortran subroutines and functions.
Fixes gh-14625