fury icon indicating copy to clipboard operation
fury copied to clipboard

Remove most pyfury pure python serializers, use pure cython only

Open chaokunyang opened this issue 1 year ago • 0 comments

Feature Request

Remove pyfury python implementation, use pure cython only

Is your feature request related to a problem? Please describe

Currently pufury has two implementation for object graph serialization:

  • pure python:
    • python/pyfury/_serializer.py
    • python/pyfury/_fury.py
  • cython based:
    • python/pyfury/_serialization.pyx

Pure python version is easy for debugging, especially for debugging complex object graph serialization, since cython code is not easy to debug.

But the code for both are very similar, thus incurred a big maintain overhead.

Describe the solution you'd like

Remove most pure python serializer, only reserve serializer dispatch framework, and keep it as the most simple way, the performance is not a requirement

Describe alternatives you've considered

No response

Additional context

No response

chaokunyang avatar Nov 24 '24 13:11 chaokunyang