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

[AIR/Tune] Deprecate mixins and replace with setup functions #27966

Closed
amogkam opened this issue Aug 17, 2022 · 0 comments · Fixed by #31295
Closed

[AIR/Tune] Deprecate mixins and replace with setup functions #27966

amogkam opened this issue Aug 17, 2022 · 0 comments · Fixed by #31295
Assignees
Labels
P1 Issue that should be fixed within a few weeks ray-team-created Ray Team created tune Tune-related issues
Milestone

Comments

@amogkam
Copy link
Contributor

amogkam commented Aug 17, 2022

We need to deprecate the wandb and mlflow mixins as they do not work with Ray AIR Trainers. However, users want the flexibility to be able to log directly from the training function.

As discussed before, for users who want this flexibility, we can provide the necessary utility functions to setup the mlflow/wandb runs

cc @xwjiang2010 @krfricke

@amogkam amogkam added this to the Ray AIR milestone Aug 17, 2022
@amogkam amogkam added tune Tune-related issues air triage Needs triage (eg: priority, bug/not-bug, and owning component) P1 Issue that should be fixed within a few weeks and removed triage Needs triage (eg: priority, bug/not-bug, and owning component) labels Aug 17, 2022
@krfricke krfricke self-assigned this Aug 17, 2022
@richardliaw richardliaw added the ray-team-created Ray Team created label Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 Issue that should be fixed within a few weeks ray-team-created Ray Team created tune Tune-related issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants