DOC: Update arctan2 docstring based on doctest output#21064
DOC: Update arctan2 docstring based on doctest output#21064rossbar merged 1 commit intonumpy:mainfrom
Conversation
rossbar
left a comment
rossbar
left a comment
There was a problem hiding this comment.
The fix LGTM, thanks @pllim . The alarming thing is that this isn't failing in numpy's tests, which indicates that we may need to take a closer look at how/whether doctest is properly picking up all the ufunc docstrings.
That's a separate issue though, thanks for the fix!
charris
changed the title
|
Hi-five on merging your first pull request to NumPy, @pllim! We hope you stick around! Your choices aren’t limited to programming – you can review pull requests, help us stay on top of new and old issues, develop educational material, work on our website, add or improve graphic design, create marketing materials, translate website content, write grant proposals, and help with other fundraising initiatives. For more info, check out: https://numpy.org/contribute |
3 participants
As per astropy/astropy#12853 (comment) . cc @mhvk and @lpsinger
p.s. We really shouldn't be picking up your doctest but since we are right now, might as well fix it?
xref scientific-python/pytest-doctestplus#174