Skip to content
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

PT: also use complete_frac for progress reporting #1698

Merged
merged 6 commits into from
Mar 4, 2025

Conversation

NeoLegends
Copy link
Member

After this PR, the torch engine will use complete_frac for progress reporting as well as num_seqs.

Example line:

ep 1 train, step 23, ctc_8 10.797, ctc_16 10.016, ce 8.758, fer 0.979, grad_norm:p2 85.874, num_seqs 34, max_size:time 239629, max_size:out-spatial 80, mem_usage:cuda:0 60.0GB, 0.926 sec/step, elapsed 0:00:55, exp. remaining 4:59:46, complete 0.31%

@NeoLegends NeoLegends self-assigned this Mar 1, 2025
@NeoLegends NeoLegends requested review from albertz and a team as code owners March 1, 2025 13:43
@albertz albertz merged commit acf09da into master Mar 4, 2025
62 checks passed
@albertz albertz deleted the moritz-report-complete-frac branch March 4, 2025 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants