You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
cloudpickle and cloudpickle_fast differ in the sense that the cloudpickle_fast's CloudPickler only has to reduce objects (the actual saving is taken care of by logic coded in the pickle.Pickler class), whereas the cloudpickle.CloudPickler reduces AND saves objects in one single method (save_{object_type}(self, obj))
Right now, the cloudpickle.CloudPickler reducing parts of the save_{object_type} methods do not use the reduce_{object_type} reducers implemented in cloudpickle_fast.
To avoid the need to write twice any change we make to a reducer, it would be great to refactor most cloudpickle.CloudPickler.save_* methods, to make them look like this
cloudpickle
andcloudpickle_fast
differ in the sense that thecloudpickle_fast
'sCloudPickler
only has toreduce
objects (the actual saving is taken care of by logic coded in thepickle.Pickler
class), whereas thecloudpickle.CloudPickler
reduces AND saves objects in one single method (save_{object_type}(self, obj)
)Right now, the
cloudpickle.CloudPickler
reducing parts of thesave_{object_type}
methods do not use thereduce_{object_type}
reducers implemented incloudpickle_fast
.To avoid the need to write twice any change we make to a reducer, it would be great to refactor most
cloudpickle.CloudPickler.save_*
methods, to make them look like thisreduce_{object_type}
being used both bycloudpickle
andcloudpickle_fast
.The text was updated successfully, but these errors were encountered: