California-Planet-Search / radvel

General Toolkit for Modeling Radial Velocity Data
http://radvel.readthedocs.io
MIT License
58 stars 53 forks source link

use dill rather than pickle #319

Closed sblunt closed 4 years ago

sblunt commented 4 years ago

dill in my experience is better able to serialize stuff than pickle, in my experience. I don't think this change breaks anything. It fixes a bug I ran into trying to serialize a NumericalPrior with a bwmethod attribute.

bjfultn commented 4 years ago

Thanks @sblunt! It looks like the Travis tests are failing. I think it might just need dill added to the dependencies.

sblunt commented 4 years ago

ok @bjfultn should be ready to go now.