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

[feat] Allow HF callback to initialize Aim run at init end #3116

Merged
merged 3 commits into from
Mar 13, 2024

Conversation

dushyantbehl
Copy link
Contributor

Current implementation of HF callback in Aim initializes the Run object at training begin time. This means if the users want to track any additional metrics they need to do so before training finishes while using HFTrainer object.

This small change which initializes the Run object on init_end makes the run object available to user of this callback via callback.experiment and allows them to track any extra metrics or parameters with the same run other than the standard metrics output by HF.

Copy link
Member

@alberttorosyan alberttorosyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @dushyantbehl. Thanks for your contribution 🙌
Changes look good!
Would you mind updating the CHANGELOG.md file according to the changes?

@alberttorosyan alberttorosyan changed the title Allow HF callback to initialize Aim run at init end. [feat] Allow HF callback to initialize Aim run at init end Mar 12, 2024
@dushyantbehl
Copy link
Contributor Author

@alberttorosyan sure!

Signed-off-by: Dushyant Behl <dushyantbehl@users.noreply.github.com>
Signed-off-by: Dushyant Behl <dushyantbehl@users.noreply.github.com>
@dushyantbehl
Copy link
Contributor Author

@alberttorosyan done with the changes, let me know if I need to change anything. 🥂

@alberttorosyan alberttorosyan merged commit 60a9f9b into aimhubio:main Mar 13, 2024
3 of 4 checks passed
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.

2 participants