Skip to content

[flex/linen]: Make rngs type "Optional" to avoid pytype errors.#1293

Merged
copybara-service[bot] merged 1 commit intomasterfrom
test_371429989
May 1, 2021
Merged

[flex/linen]: Make rngs type "Optional" to avoid pytype errors.#1293
copybara-service[bot] merged 1 commit intomasterfrom
test_371429989

Conversation

@copybara-service
Copy link
Copy Markdown

@copybara-service copybara-service bot commented May 1, 2021

[flex/linen]: Make rngs type "Optional" to avoid pytype errors.

Setting rngs to None in apply() is valid (also the default), but setting it
explicitly in a callee site causes pytype errors.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 1, 2021

Codecov Report

Merging #1293 (c69c6b3) into master (2faf0ea) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1293   +/-   ##
=======================================
  Coverage   82.31%   82.31%           
=======================================
  Files          65       65           
  Lines        5310     5310           
=======================================
  Hits         4371     4371           
  Misses        939      939           
Impacted Files Coverage Δ
flax/linen/module.py 95.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2faf0ea...c69c6b3. Read the comment docs.

Setting `rngs` to None in apply() is valid (also the default), but setting it
explicitly in a callee site causes pytype errors.

PiperOrigin-RevId: 371438792
@copybara-service copybara-service bot merged commit 0848d9c into master May 1, 2021
@copybara-service copybara-service bot deleted the test_371429989 branch May 1, 2021 01:27
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.

2 participants