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

[Feature] add_scalar in logger_hook for after_test_epoch #1582

Open
AwePhD opened this issue Oct 5, 2024 · 0 comments
Open

[Feature] add_scalar in logger_hook for after_test_epoch #1582

AwePhD opened this issue Oct 5, 2024 · 0 comments

Comments

@AwePhD
Copy link

AwePhD commented Oct 5, 2024

What is the feature?

Hello,

Why the method Visualizer.add_scalars is only called in after_val_epoch and not it after_test_epoch?

I think it can be a good thing to add this feature for test with some vis_backends for logs tracking (WandB, MLFlow, Tensorboard ...)

I made the edit locally of the after_test_epoch, I use a new attribute for the LoggerHook named log_test_metric_by_visualizer set by default to True.

What do you think about it? Would it be better to this in another hook? (I think it would be a bit weird since the logger hook already those kind of loggings.

Any other context?

No response

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

No branches or pull requests

1 participant