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

Add cythonized udunits2 #120

Merged
merged 31 commits into from
Aug 6, 2020
Merged

Add cythonized udunits2 #120

merged 31 commits into from
Aug 6, 2020

Conversation

mcflugen
Copy link
Member

@mcflugen mcflugen commented Aug 2, 2020

This pull request cythonizes the udunits2 library as pymt.udunits2. This will, hopefully, provide a more robust unit conversion module that works on multiple platforms and is under our control (we were having issues with cfunits). Thus far, our udunits2 module may not have all of the functionality of cfunits, but it's enough for our purpose at the moment.

Still to do:

  • Ensure all the tests are passing and the new module is well tested
  • Add documentation for the new module
  • Deploy a binary distribution to PyPI as part of our CI
  • Remove the old pymt.cfunits package

@mcflugen mcflugen merged commit d3e936e into master Aug 6, 2020
@mcflugen mcflugen deleted the mcflugen/add-cythonized-udunits2 branch August 6, 2020 05:29
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.

1 participant