Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[export] Refactor the imports for the public API of jax.experimental.export #174

Closed
wants to merge 0 commits into from

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Dec 15, 2023

[export] Refactor the imports for the public API of jax.experimental.export

Previously we used from jax.experimental.export import export and
export.export(fun). Now we want to add the public API directly
to jax.experimental.export, for the following desired usage:

from jax.experimental import export

exp: export.Exported = export.export(fun)
ser: bytearray = export.serialize(exp)
exp1: export.Exported = export.deserialized(ser)
export.call(exp1)

This change requires changing the type of
jax.experimental.export.export from a
module to a function. This confuses
pytype for the targets with strict type checking,
which is why I attempt to make this change
atomically throughout the internal code base.

In order to support backwards compatibility with
OSS packages, this change also includes explicit
JAX version checks in several OSS packages, and
also adds to the export function the attributes
that the old export module had.

@copybara-service copybara-service bot force-pushed the test_591228914 branch 13 times, most recently from b3b4332 to 52212a7 Compare December 19, 2023 07:46
@copybara-service copybara-service bot force-pushed the test_591228914 branch 2 times, most recently from 7d250b7 to bd1f0bc Compare January 8, 2024 13:05
@copybara-service copybara-service bot closed this Jan 8, 2024
@copybara-service copybara-service bot deleted the test_591228914 branch January 8, 2024 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants