Skip to content

DOC: clarify cdouble constructor docstring - #32847

Open
gregwb wants to merge 1 commit into
numpy:mainfrom
gregwb:complex-docstring-update
Open

gregwb wants to merge 1 commit into
numpy:mainfrom
gregwb:complex-docstring-update

Conversation

@gregwb

@gregwb gregwb commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Clarify that cdouble, unlike Python complex, does not accept keyword arguments.

PR summary

Re-applies #32795, minus the clongdouble change, which I'll open another PR to address

First time contributor introduction

Hello! I'm Greg, engineer at Bloomberg participating in the NumFOCUS Sustaining Open Source Series.

@InessaPawson FYI sustain-2026!

AI Disclosure

I used OpenAI Codex used to help inspect and explain to me the relevant parts of the codebase,.

Clarify that cdouble, unlike Python complex, does not
accept keyword arguments.
@jorenham

jorenham commented Oct 1, 2026

Copy link
Copy Markdown
Member

We don't need to do this after #32848, right?

@gregwb

gregwb commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

We don't need to do this after #32848, right?

They are independent PRs.
This one is a documentation change to clarify existing behaviour of cdouble (that it is not completely “compatible” with the python’s complex type with respect to accepting kwargs)

#32848, on the other hand fixes clongdouble
behaviour.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants