-
Notifications
You must be signed in to change notification settings - Fork 22
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
Adding tensorboard tracking/visualization and nnx display example #118
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, you do not want to showcase tbX in addition to pure tb/tf ?
Yea, I had mixed thoughts on this - I believe we were trying to stick as closely to the jax-ai-stack modules as possible and tbX wouldn't be in there, even if it is a good extra take on the API - I can get in a quick "and this is what it would look like in tbX" couple cells if we think it's an improvement and not taking away from the specific stack attention. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Just a couple comments below.
Looks good! We just need to resolve conflicts and then we can merge this! |
@jakevdp conflicts resolved |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
…x-ml#118) * adding tensorboard tracking/visualization and nnx display example * Address Victor's initial feedback for information and links * Addressing feedback - correcting links and final code cell
Starting from basic model/data in getting_started_with_jax_for_AI and modifying as appropriate to showcase tensorboard integration/tracking as well as basic nnx.display examples.