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

Deprecate jax.experimental.loops, step 2. #10278

Closed
wants to merge 0 commits into from
Closed

Conversation

copybara-service[bot]
Copy link

Deprecate jax.experimental.loops, step 2.

Add deprecation warning and remove the tests.

@gnecula gnecula self-assigned this Apr 14, 2022
@copybara-service copybara-service bot force-pushed the test_441682387 branch 3 times, most recently from 6eae300 to 65016cf Compare April 14, 2022 08:07
@copybara-service copybara-service bot force-pushed the test_441682387 branch 2 times, most recently from 63ff96f to fb9e00a Compare April 14, 2022 18:44
@copybara-service copybara-service bot closed this Apr 14, 2022
@copybara-service copybara-service bot deleted the test_441682387 branch April 14, 2022 19:39
gnecula added a commit to gnecula/jax that referenced this pull request Jul 25, 2022
Has been deprecated since April 2022.
See issue google#10278 for an alternative API.
gnecula added a commit to gnecula/jax that referenced this pull request Jul 26, 2022
Has been deprecated since April 2022.
See issue google#10278 for an alternative API.
@wookayin
Copy link
Contributor

Arrived from the deprecation noice (changelog) -- what would be an alternative API?

@gnecula
Copy link
Collaborator

gnecula commented Aug 29, 2022

I am not sure how the PR got lost, but the commit that added the deprecation instructions is here.

In retrospect, it was a bad idea to add the deprecation instructions in loops.py, which was then removed a few months later.

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.

None yet

2 participants