-
-
Notifications
You must be signed in to change notification settings - Fork 648
[skip ci] Update MNIST_on_TPU.ipynb #2252
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
Conversation
Looks good ! Thanks @FarehaNousheen Notebook can be checked here: |
@FarehaNousheen could you please also add missing f-strings: pbar.log_message(
- "Training Results - Epoch: {engine.state.epoch} Avg accuracy: {avg_accuracy:.2f} Avg loss: {avg_nll:.2f}"
+ f"Training Results - Epoch: {engine.state.epoch} Avg accuracy: {avg_accuracy:.2f} Avg loss: {avg_nll:.2f}"
) and print(
- "Validation Results - Epoch: {engine.state.epoch} Avg accuracy: {avg_accuracy:.2f} Avg loss: {avg_nll:.2f}"
+ f"Validation Results - Epoch: {engine.state.epoch} Avg accuracy: {avg_accuracy:.2f} Avg loss: {avg_nll:.2f}"
) |
Sure I'll.
…On Thu, Oct 7, 2021 at 2:16 AM vfdev ***@***.***> wrote:
@FarehaNousheen <https://github.com/FarehaNousheen> could you please also
add missing f-strings:
pbar.log_message(
f"Training Results - Epoch: {engine.state.epoch} Avg accuracy: {avg_accuracy:.2f} Avg loss: {avg_nll:.2f}"
)
and
print(
f"Validation Results - Epoch: {engine.state.epoch} Avg accuracy: {avg_accuracy:.2f} Avg loss: {avg_nll:.2f}"
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2252 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVNZ23RIPEHDQB63DIKFD7LUFSYTFANCNFSM5FPMZUBA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
[image: photo]
Fareha Nousheen
M 9049800892 <+9049800892>
E ***@***.*** ***@***.***>
W www.hubsab.com <http://+www.hubsab.com>
<http://www.facebook.com/fareha.nousheen>
<http://www.linkedin.com/in/fareha-nousheen-913132170/>
<http://twitter.com/FarehaNousheen> <http://www.instagram.com/hubsab_2018/>
Create your own email signature
<https://www.wisestamp.com/signature-in-email?utm_source=promotion&utm_medium=signature&utm_campaign=create_your_own>
|
Hi @FarehaNousheen thanks for the changes. |
ok, doing it now. Actually, after committing the changes some error
occurred. Checking it now.
…On Fri, Oct 8, 2021 at 12:58 AM Ishan Kumar ***@***.***> wrote:
Hi @FarehaNousheen <https://github.com/FarehaNousheen> thanks for the
changes.
I think your latest commit has messed up the notebook JSON format. I am
not able to open it here and locally.
It might have been caused due to making edits in the raw format, to fix it
you can remove the changes in the last commit and then update the text
after opening as a Jupyter notebook.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2252 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVNZ23U2W4MWGDOJIHI74TDUFXYEFANCNFSM5FPMZUBA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
[image: photo]
Fareha Nousheen
M 9049800892 <+9049800892>
E ***@***.*** ***@***.***>
W www.hubsab.com <http://+www.hubsab.com>
<http://www.facebook.com/fareha.nousheen>
<http://www.linkedin.com/in/fareha-nousheen-913132170/>
<http://twitter.com/FarehaNousheen> <http://www.instagram.com/hubsab_2018/>
Create your own email signature
<https://www.wisestamp.com/signature-in-email?utm_source=promotion&utm_medium=signature&utm_campaign=create_your_own>
|
d535935
to
03ec7c2
Compare
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 @FarehaNousheen !
The notebook is checked on google collab. Sharing the link below for further step. |
Fixes #2239
Description: Update Install links for PyTorch XLA in MNIST TPU
Check list: