| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
Some of these look "ok," but others look like automatic replacements that don't belong--I've pointed out a few from the latter category from a quick scan.
Sorry, something went wrong.
|
The f2c stuff is generated and should be left alone. |
Sorry, something went wrong.
Okay, reverted. |
Sorry, something went wrong.
|
This error seems erroneous to me: E AssertionError:
E Arrays are not equal
E
E Mismatched elements: 2 / 10 (20%)
E Max absolute difference among violations: 1208103775
E Max relative difference among violations: 32749.
E ACTUAL: array([[1208103775, 32750, 2, 3, 4],
E [ 5, 6, 7, 8, 9]],
E dtype=int32)
E DESIRED: array([[0, 1, 2, 3, 4],
E [5, 6, 7, 8, 9]])
POINTER = <built-in function POINTER>
a = array([[ -217879408, 32745, -1231388384, 32745, 4],
[ 5, 6, 7, 48, 0]],
dtype=int32)
numpy/tests/test_ctypeslib.py:218: AssertionError
That does not seem related to this PR. I expect re-running the tests would make them pass again. (Sorry if I am wrong. This is just my guess.) If that IS the case, perhaps we should open a bug ticket for intermittently failing CI. |
Sorry, something went wrong.
|
You can ignore the PyPy failure, it is a known bug in PyPy. |
Sorry, something went wrong.
There was a problem hiding this comment.
Changing the error messages is technically a backward compatibility issue - can you add a release note for the off chance someone is using the error message in a test like the numpy tests were?
Sorry, something went wrong.
I have added a release note. I tried to word it in keeping with other release notes. But feel free to holler if you'd like it worded differently. |
Sorry, something went wrong.
|
There was a spelling error in the release note 🙃 so I fixed that and made it a little less wordy. Will merge shortly. |
Sorry, something went wrong.
|
Actually I think @tylerjereddy needs to give it another pass because of branch protection |
Sorry, something went wrong.
| ============ | ||
|
|
||
| Use delimiter rather than delimitor as kwarg in mrecords | ||
| Use delimiter rather than delimiter as kwarg in mrecords |
There was a problem hiding this comment.
I think this still needs to be reverted?
Sorry, something went wrong.
There was a problem hiding this comment.
Reverted
Sorry, something went wrong.
I think I found one more case that should be reverted, but I've dismissed the blocking review apart from that. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
This is just a simple cleanup PR. I found various spelling errors and fixed them.
And then I went hunting for spelling errors and found a few more.
I have not changed any variable names here, this is all just docs and docstrings. So this should be a very safe change.