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

Have added basic config to output SR outputs for val sets to Tensorboard #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

grofit
Copy link

@grofit grofit commented Jan 30, 2022

This change will allow users to add the configuration variables to the logger section to define tb_log_generated: true and tb_log_lr: true which will then output the same validation images to tensorboard directly for users.

This was something I added purely for my own benefit when training pixel based stuff, and it could be HUGE outputs with loads of validation sets, however there seemed like no sane way to allow users to dictate what validation sets they should output, I thought maybe a sort of image_log_skip variable that would let a user specify they only wanted to write out like 1 in every 50 validation images etc.

Anyway feel free to reject, its something I did for BasicSR and now I am doing a little prototype with this newer version I wanted to be able to report my stuff all in TB, so make any alterations you want.

Thanks again for the great library.

This change will allow users to add the configuration variables to the `logger` section to define `tb_log_generated`: true and `tb_log_lr`: true which will then output the same validation images to tensorboard directly for users.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant