Suggestion: More user-friendly labels in "Generate Configuration" #29

New Issue

sakharam_gatne · 2023-02-23T17:57:53Z

sakharam_gatne commented

2023-02-23 17:57:53 +00:00

The Print Frequency per Epoch and Save Frequency per Epoch could probably be changed to something like Print (& save) every 'n' epochs.

Maybe I'm just an idiot, but I thought it meant how many times to print/save "per epoch". So for a dataset of about 1500 audio files, I set both values to 50, and kept wondering why there was no checkpoint saved after two hours of training...

The `Print Frequency per Epoch` and `Save Frequency per Epoch` could probably be changed to something like `Print (& save) every 'n' epochs`. ![image](/attachments/18d4b75d-373e-44ea-a3c8-ff7b4e7a6ac1) Maybe I'm just an idiot, but I thought it meant how many times to print/save "per epoch". So for a dataset of about 1500 audio files, I set both values to 50, and kept wondering why there was no checkpoint saved after two hours of training...

image.png

9.1 KiB

mrq commented

2023-02-23 20:12:41 +00:00

No you're right, it implies it'll do it within an epoch, rather than every epoch. desu it was originally labeled "Print Frequency" and "Save Frequency" when it was just in terms of iterations, but my brainworms neglected to consider the implications when I slapped the unit on it.

I'll reword it whenever I get a chance.

No you're right, it implies it'll do it within an epoch, rather than every epoch. desu it was originally labeled "Print Frequency" and "Save Frequency" when it was just in terms of iterations, but my brainworms neglected to consider the implications when I slapped the unit on it. I'll reword it whenever I get a chance.

mrq commented

2023-02-23 23:24:12 +00:00

Remedied in commit 1e0fec4358. I renamed it to "Print/Save Frequency (in epochs)".

There's no way it's only been three hours since; it felt like an eternity since I was able to get around to this.

Remedied in commit 1e0fec4358a11bac2cf20cef05d3e890638ee7c5. I renamed it to "Print/Save Frequency (in epochs)". There's no way it's only been three hours since; it felt like an eternity since I was able to get around to this.

mrq closed this issue

2023-02-23 23:24:12 +00:00

Sign in to join this conversation.