Skip to content

log time per epoch#308

Merged
wiederm merged 2 commits intomainfrom
dev-log-epoch-time
Nov 5, 2024
Merged

log time per epoch#308
wiederm merged 2 commits intomainfrom
dev-log-epoch-time

Conversation

@wiederm
Copy link
Copy Markdown
Member

@wiederm wiederm commented Nov 5, 2024

Pull Request Summary

This PR adds logging for epoch time. The time is measured as wall time between on_train_epoch_start and on_train_epoch_end method calls.

@wiederm wiederm self-assigned this Nov 5, 2024
@wiederm wiederm added the enhancement New feature or request label Nov 5, 2024
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 5, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 76.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 85.41%. Comparing base (2a6b208) to head (d0666b7).
Report is 3 commits behind head on main.

Additional details and impacted files

@wiederm wiederm merged commit c1430f2 into main Nov 5, 2024
@wiederm wiederm deleted the dev-log-epoch-time branch November 5, 2024 09:59
This was referenced Nov 7, 2024
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@wiederm wiederm

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wiederm @codecov-commenter