Skip to content

Test pickle support #1481

Open
Open
@jheek

Description

@jheek

PR #1475 fixes cloudpickle support but doesn't support add any testing.

Our current test setup seems to import dill indirectly which is incompatible with cloudpickle. Ideally we want to setup testing such that both are tested but don't get in the way of each other. Also we will need to think about how we can test that internal Flax state isn't serialized like was previously the case.

Metadata

Metadata

Assignees

Labels

Priority: P2 - no scheduleBest effort response and resolution. We have no plan to work on this at the moment.Status: pull requests welcomeWe agree with the direction proposed, feel free to give it a shot and file a pull request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions