-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
PyTorch Ignite example #4487
PyTorch Ignite example #4487
Conversation
for more information, see https://pre-commit.ci
Codecov Report
@@ Coverage Diff @@
## master #4487 +/- ##
=======================================
Coverage 82.98% 82.98%
=======================================
Files 319 319
Lines 16968 16968
=======================================
Hits 14081 14081
Misses 2887 2887 Continue to review full report at Codecov.
|
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.
Only added some comments. Don't know how to procede.
Co-authored-by: Matthias Fey <matthias.fey@tu-dortmund.de>
for more information, see https://pre-commit.ci
Any news on this ? |
Will try to merge in the upcoming week. Sorry for the delay! |
Ok perfect, don't worry. I was just checking in the case this was forgotten. |
Sorry again for the delay. I currently do not have access to a GPU and still want to run it before merging. |
Ok, no worries. Let me know !
Il ven 20 mag 2022, 05:53 Matthias Fey ***@***.***> ha
scritto:
… Sorry again for the delay. I currently do not have access to a GPU and
still want to run it before merging.
—
Reply to this email directly, view it on GitHub
<#4487 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADGSCHOP4YAQVRJTHOLNVOTVK4EEDANCNFSM5TSEYTGA>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
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.
Thank you!
@rusty1s I created the pull request connected to issue #4454 . This is my example. We can work together to adjust the content, the style and the amount of comments. For now it includes a lot of things. Maybe tensorboard logging could be removed in order to make it more simple.
Let me know if you like it.
And thanks for the explanation about how to make the pull request.