Skip to content

Conversation

@peytondmurray
Copy link
Contributor

No description provided.

@ngoldbaum ngoldbaum merged commit 914afe8 into main Dec 12, 2022
Py_INCREF(given_descrs[1]);
loop_descrs[1] = given_descrs[1];
return NPY_NO_CASTING;
return NPY_SAFE_CASTING;
Copy link
Member

Choose a reason for hiding this comment

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

You know that you don't have any byte-swaps or so, so this is always NO_CASTING. In principle, you could always return the singleton instance also. (The only reason NumPy has more instances is byte-swapping and metadata, which you don't have to support.)

Bu it's fine to just delete it for this DType really.

The function currently doesn't fill in the output loop_descrs[2]?

@ngoldbaum
Copy link
Member

If you'd like, please feel free to add some tests for quaddtype and then hook that up to the github actions workflow. I merged this quickly (before Sebastian had time to comment..) but if that's not what you want please let me know.

@peytondmurray peytondmurray deleted the fix-quaddtype-debug-symbols branch December 12, 2022 21:56
@peytondmurray
Copy link
Contributor Author

No problem, and thanks for the feedback to you both. I'm still filling out the implementation, so I'll definitely add tests for this now.

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.

4 participants