amazon-ion / ion-python

A Python implementation of Amazon Ion.
https://amazon-ion.github.io/ion-docs/
Apache License 2.0
253 stars 50 forks source link

Add C timestamp implementation; need to fix tests #324

Closed nirosys closed 3 months ago

nirosys commented 6 months ago

Issue #, if available: n/a

Description of changes: This PR adds a C implementation for Timestamps, rather than using the python implementation found in amazon.ion.core.

Drafting because there are still a couple of things I need to address:

Had some issues locally with py.test and what looked like incredibly large decimals, but I'm not sure if that's due to write-side issue above.

I'll add a more detailed description of the changes later today.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

nirosys commented 6 months ago

Will address the conflict, with incoming fixes as well.

nirosys commented 3 months ago

Closing this for now. Improvements were made with #337.