Skip to content

Add UnpicklingError import for dill.load_session() to fix #648#649

Merged
mmckerns merged 1 commit intouqfoundation:masterfrom
lizy14:fix-UnpicklingError-not-defined
Feb 24, 2024
Merged

Add UnpicklingError import for dill.load_session() to fix #648#649
mmckerns merged 1 commit intouqfoundation:masterfrom
lizy14:fix-UnpicklingError-not-defined

Conversation

@lizy14
Copy link
Contributor

@lizy14 lizy14 commented Feb 22, 2024

Summary

Fix #648

Checklist

Documentation and Tests

  • Added relevant tests that run with python tests/__main__.py, and pass.
  • Added relevant documentation that builds in sphinx without error.
  • Added new features that are documented with examples.
  • Artifacts produced with the main branch work as expected under this PR.

Release Management

  • Added "Fixes #NNN" in the PR body, referencing the issue (#NNN) it closes.
  • Added a comment to issue #NNN, linking back to this PR.
  • Added rationale for any breakage of backwards compatibility.
  • Requested a review.

Copy link
Member

@mmckerns mmckerns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mmckerns mmckerns merged commit 70f569b into uqfoundation:master Feb 24, 2024
@mmckerns mmckerns added this to the dill-0.3.9 milestone Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dill.load_session() raises NameError: name 'UnpicklingError' is not defined

2 participants