Skip to content

Conversation

joaopfg
Copy link
Contributor

@joaopfg joaopfg commented Jul 7, 2022

Add docs about the C-API incompatibility error in the file troubleshooting-importerror.rst and pointing to this docs in the error message generated at the submodule core.code_generators.generate_numpy_api .

Related issue

@joaopfg joaopfg changed the title ENH: Add docs about the C-API incompatibility error in the file troubleshooting-importerror.rst and pointing to this docs in the error message generated at the submodule core.code_generators.generate_numpy_api ENH: Add docs about the C-API incompatibility error iand pointing to this docs in the error message Jul 7, 2022
@joaopfg joaopfg changed the title ENH: Add docs about the C-API incompatibility error iand pointing to this docs in the error message ENH: Add docs about the C-API incompatibility error and pointing to this docs in the error message Jul 7, 2022
C-API incompatibility
---------------------------

If you see this error you may have:
Copy link
Member

Choose a reason for hiding this comment

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

Please add at least some of the text of the error message. Also the lines below should be reflowed to not exceed ~80 columns

Copy link
Contributor Author

@joaopfg joaopfg Jul 11, 2022

Choose a reason for hiding this comment

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

@mattip Thanks for help. I updated the PR. Is it ok now ?

@mattip
Copy link
Member

mattip commented Jul 8, 2022

Fixes #21901

@charris charris changed the title ENH: Add docs about the C-API incompatibility error and pointing to this docs in the error message DOC, MAINT: Document the C-API incompatibility error and point the error message to this doc. Jul 10, 2022
@mattip mattip merged commit c856349 into numpy:main Jul 12, 2022
@mattip
Copy link
Member

mattip commented Jul 12, 2022

Thanks @joaopfg

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

Successfully merging this pull request may close these issues.

2 participants