remove vendored cloudpickle

ubarsc / rios

A raster processing layer on top of GDAL

https://www.rioshome.org

GNU General Public License v3.0

14 stars 7 forks source link

remove vendored cloudpickle #63

Closed gillins closed 1 year ago

gillins commented 1 year ago

Fixes #62

neilflood commented 1 year ago

Seems fine to me, thank you.

gillins commented 1 year ago

Hmm, seems there is a difference with the non-vendored one...

  File "/usr/local/lib/python3.10/dist-packages/rios/parallel/aws/batch.py", line 181, in startOneJob
    allInputsPickled = cloudpickle.dumps(allInputs)
AttributeError: module 'cloudpickle.cloudpickle' has no attribute 'dumps'

neilflood commented 1 year ago

That's weird. It seems it is supposed to be there, as per the current docs https://pypi.org/project/cloudpickle/ Looks like the import is a bit different?

gillins commented 1 year ago

Yep should be just import cloudpickle, weird we might have had this wrong for people with it installed since forever...