Skip to content

Use Python 3 CAPI features to cleanup PRINTPYOBJERR #19106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
pearu opened this issue May 26, 2021 · 1 comment
Closed

Use Python 3 CAPI features to cleanup PRINTPYOBJERR #19106

pearu opened this issue May 26, 2021 · 1 comment
Assignees
@HaoZeke
Labels
03 - Maintenance component: numpy.f2py

Comments

@pearu
Copy link
Contributor

pearu commented May 26, 2021

While you're here, we could clean this up using Python 3 CAPI features to

   PyErr_SetFormat(#modulename#_error, \"try_pyarr_from_string failed: %R\", obj);

but maybe cleaning up all the PRINTPYOBJERR calls is best left to another PR

Originally posted by @eric-wieser in #18759 (comment)
@pearu pearu mentioned this issue May 26, 2021
Merged
@HaoZeke HaoZeke self-assigned this Aug 26, 2021
@HaoZeke
Copy link
Member

HaoZeke commented Dec 18, 2023

Closing for now, there are cleanups which would make sense in the error prints (e.g. I'd like a base exception for all F2PY generated code) but this doesn't seem to be an actual part of the API..

@HaoZeke HaoZeke closed this as completed Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HaoZeke HaoZeke

Labels
03 - Maintenance component: numpy.f2py
Projects
f2py core
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pearu @HaoZeke