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

remove cell_size docs from RNN #3186

Merged
merged 1 commit into from
Jul 7, 2023
Merged

remove cell_size docs from RNN #3186

merged 1 commit into from
Jul 7, 2023

Conversation

cgarciae
Copy link
Collaborator

@cgarciae cgarciae commented Jul 7, 2023

What does this PR do?

  • Remove the cell_size entry from the RNN docstring.

@cgarciae cgarciae requested a review from chiamp July 7, 2023 16:00
@codecov-commenter
Copy link

codecov-commenter commented Jul 7, 2023

Codecov Report

Merging #3186 (aac2f05) into main (1e7a8b1) will decrease coverage by 0.01%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main    #3186      +/-   ##
==========================================
- Coverage   82.30%   82.30%   -0.01%     
==========================================
  Files          54       54              
  Lines        6054     6059       +5     
==========================================
+ Hits         4983     4987       +4     
- Misses       1071     1072       +1     
Impacted Files Coverage Δ
flax/linen/normalization.py 96.71% <80.00%> (-0.61%) ⬇️
flax/linen/recurrent.py 97.28% <100.00%> (+0.01%) ⬆️

@copybara-service copybara-service bot merged commit eb1c5ee into main Jul 7, 2023
19 checks passed
@copybara-service copybara-service bot deleted the fix-rnn-docs branch July 7, 2023 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants