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.

@SwayamInSync SwayamInSync left a comment

Copy link
Copy Markdown
Member

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