Skip to content
Snippets Groups Projects

Remove dynesty from __getstate__

Merged Colm Talbot requested to merge pickle-dynesty into master

This came from an external request to make our Dynesty object picklable. Here the external_sampler (which is the dynesty module) is stored in the object and can't be pickled by pickle.

There may be other neater fixes, if people have thoughts, let me know.

Merge request reports

Pipeline #65359 passed

Pipeline passed for cabdf287 on pickle-dynesty

Test coverage 69.00% (0.00%) from 1 job
Approved by

Merged by Gregory AshtonGregory Ashton 5 years ago (Jun 12, 2019 2:45am UTC)

Merge details

  • Changes merged into master with 0ccf55c7 (commits were squashed).
  • Deleted the source branch.

Pipeline #66472 passed with warnings

Pipeline passed with warnings for 0ccf55c7 on master

Test coverage 69.00% (0.00%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading