Skip to content

Add default zero constructor for QuadPrecision#84

Merged
SwayamInSync merged 5 commits into
numpy:mainfrom
ejeffrey:add_default_constructor
May 9, 2026
Merged

Add default zero constructor for QuadPrecision#84
SwayamInSync merged 5 commits into
numpy:mainfrom
ejeffrey:add_default_constructor

Conversation

@ejeffrey
Copy link
Copy Markdown
Contributor

Most python numeric types have default constructors, and pandas relies on it when building string representations of data frames

Most python numeric types have default constructors, and pandas
relies on it when building string representations of data frames
@ejeffrey
Copy link
Copy Markdown
Contributor Author

Fixes #83

@ngoldbaum
Copy link
Copy Markdown
Member

Looks like you need to declare Pandas as a test dependency.

@ejeffrey
Copy link
Copy Markdown
Contributor Author

Good point. I think I added that in the right place now.

@ngoldbaum
Copy link
Copy Markdown
Member

It looks like there are still failures on the tests for older CPUs and more obscure architectures. Sorry this is so tricky!

I think you might need to use importorskip to skip the test if pandas isn't importable.

@ngoldbaum
Copy link
Copy Markdown
Member

btw, you can enable actions on your fork so you don't need to wait on me to hit the approve button over here

@ejeffrey
Copy link
Copy Markdown
Contributor Author

It turns out the test dependencies weren't being installed in some of the tests. I fixed that and now it's passing on my repo except for the build docs job which looks like it's missing an access token.

Copy link
Copy Markdown
Member

@SwayamInSync SwayamInSync left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just added some small nits and test to cover longdouble backend.
Thanks @ejeffrey LGTM!!

@SwayamInSync
Copy link
Copy Markdown
Member

CI is all green, merging this 🚀

@SwayamInSync SwayamInSync merged commit 2b63d54 into numpy:main May 9, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants