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

Fix tracking context warnings #90

Open
mwartell opened this issue Dec 19, 2023 · 1 comment
Open

Fix tracking context warnings #90

mwartell opened this issue Dec 19, 2023 · 1 comment
Assignees

Comments

@mwartell
Copy link
Contributor

We get piles of

2023-12-19 13:39:19 2m37s WARNING  charmory.track:_wrapper:177 Parameters with prefix PyTorchClassifier have already been logged and will be overwritten. Use a unique prefix or start a new tracking context with `tracking_context` to avoid this warning.

warnings. Find what that complaint means and fix it.

@treubig26
Copy link
Collaborator

This is happening because when track_init_params gets used multiple times for the same class, the class's __init__ method gets wrapped multiple times, resulting in this warning. track_init_params should detect that the class has already been instrumented, and not re-wrap the __init__ method.

@deprit deprit changed the title fix tracking_context warnings Fix tracking context warnings Aug 17, 2024
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

2 participants