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

Better decoupling from flaml #238

Open
baggiponte opened this issue Jun 11, 2024 · 0 comments
Open

Better decoupling from flaml #238

baggiponte opened this issue Jun 11, 2024 · 0 comments
Labels
build Build system and dependencies priority: high
Milestone

Comments

@baggiponte
Copy link
Collaborator

baggiponte commented Jun 11, 2024

We should be installing flaml[automl] instead. Since it depends on pandas and some other libraries, it should be made a separate dependency group. However this requires quite a lot of refactoring:

  1. Not straightforward:
    class FLAMLRegressor:
    1. Requires moving this forecaster somewhere else:
      class flaml_lightgbm(Forecaster):
  2. Easy:
    from flaml import tune
@baggiponte baggiponte added priority: high build Build system and dependencies labels Jun 11, 2024
@baggiponte baggiponte added this to the 0.9.X milestone Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build system and dependencies priority: high
Projects
None yet
Development

No branches or pull requests

1 participant