-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Apply the ignore of the save_hyperparameters
function to args
#19761
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
Labels
feature
Is an improvement or enhancement
good first issue
Good for newcomers
help wanted
Open to be worked on
Comments
save_hyperparameters
function to args
I'll take this up. |
hey did you resolve this issue? |
I'll take this up. |
Hey! Is this issue still open? I checked the PR #20213 and it's still open. I'd love to contribute if it's still open. |
Hi @doveppp, it already filters. ![]() We don't need to again filter it (as per your suggestion). |
Merged
7 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature
Is an improvement or enhancement
good first issue
Good for newcomers
help wanted
Open to be worked on
Description & Motivation
Currently, the ignore parameter of the
save_hyperparameters
function only filters its own arguments.Can we apply
hp = {k: v for k, v in hp.items() if k not in ignore}
beforeobj._set_hparams(hp)
?Pitch
No response
Alternatives
No response
Additional context
No response
cc @Borda
The text was updated successfully, but these errors were encountered: