Skip to content
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.

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

MAINT: port remaining C modules to Cython in climate, core and timeseries extension modules #230

Open
fkuehlein opened this issue Jul 2, 2024 · 0 comments
Labels
maintenance something should be improved or is outdated

Comments

@fkuehlein
Copy link
Collaborator

There are few remaining functions in those extension modules that are still implemented in C in the respective src_numerics.c files. Porting them to Cython would reduce language diversity of the package and facilitate debugging. Whats more, previous issues listed below have shown that the ports from the legacy weave.inline structure contained errors and should be thoroughly reviewed. This would also help to isolate issues with C compilation, such as #225.

See CHANGELOG.rst for examples:

@fkuehlein fkuehlein added the maintenance something should be improved or is outdated label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance something should be improved or is outdated
Projects
None yet
Development

No branches or pull requests

1 participant