Skip to content

Update landing page example #2366

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

Merged
merged 1 commit into from
Oct 10, 2022

Conversation

cgarciae
Copy link
Collaborator

@cgarciae cgarciae commented Aug 3, 2022

What does this PR do?

Fixes #2332

  • Improves the landing page example

@cgarciae cgarciae added the Priority: P2 - no schedule Best effort response and resolution. We have no plan to work on this at the moment. label Sep 1, 2022
@cgarciae cgarciae force-pushed the improve-landing-page-example branch from 39f1aca to f315721 Compare September 19, 2022 22:56
@cgarciae cgarciae marked this pull request as ready for review September 19, 2022 22:56
@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2022

Codecov Report

Merging #2366 (c62c428) into main (b8d1162) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2366      +/-   ##
==========================================
- Coverage   79.08%   78.98%   -0.11%     
==========================================
  Files          49       49              
  Lines        5188     5139      -49     
==========================================
- Hits         4103     4059      -44     
+ Misses       1085     1080       -5     
Impacted Files Coverage Δ
flax/core/nn/linear.py 42.34% <0.00%> (-0.52%) ⬇️
flax/struct.py 78.12% <0.00%> (-0.34%) ⬇️
flax/core/scope.py 89.72% <0.00%> (-0.07%) ⬇️
flax/core/lift.py 95.81% <0.00%> (-0.07%) ⬇️
flax/linen/normalization.py 97.10% <0.00%> (-0.05%) ⬇️
flax/core/frozen_dict.py 96.19% <0.00%> (-0.04%) ⬇️
flax/traverse_util.py 98.52% <0.00%> (-0.01%) ⬇️
flax/linen/summary.py 99.00% <0.00%> (-0.01%) ⬇️
flax/linen/recurrent.py 100.00% <0.00%> (ø)
flax/training/train_state.py 0.00% <0.00%> (ø)
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

y = model.apply(variables, x) # make forward pass
x = jnp.empty((4, 28, 28, 1)) # fake data
variables = model.init(PRNGKey(42), x) # initialize weights
y = model.apply(variables, x) # make forward pass
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: To make the grammar sound a bit better, maybe

Suggested change
y = model.apply(variables, x) # make forward pass
y = model.apply(variables, x) # forward propagate the input

@cgarciae cgarciae force-pushed the improve-landing-page-example branch from 3558870 to c62c428 Compare October 10, 2022 18:40
@cgarciae
Copy link
Collaborator Author

@8bitmp3 thanks for the feedback! Made the most of the changes except I left "make forward pass" as it sounded better in my head (feel free to push back).

@copybara-service copybara-service bot merged commit d493d44 into google:main Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: P2 - no schedule Best effort response and resolution. We have no plan to work on this at the moment. pull ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Landing Page example
4 participants