Support logger_formatter without automl dependencies #1356

thinkall · 2024-09-21T09:12:15Z

Why are these changes needed?

To fix #1355

Closes #1355

I've used pre-commit to lint the changes in this PR (note the same in integrated in our CI checks).
I've included any doc changes needed for https://microsoft.github.io/FLAML/. See https://microsoft.github.io/FLAML/docs/Contribute#documentation to build and test documentation locally.
I've added tests (if relevant) corresponding to the changes introduced in this PR.
I've made sure all auto checks have passed.

ianhelle · 2024-09-23T15:56:48Z

Yep. Thanks @thinkall - that fixed the issue

Support logger_formatter without automl dependencies

0e07087

thinkall mentioned this pull request Sep 21, 2024

[Bug]: Importing xgboost without it being marked as mandatory dependency #1355

Closed

thinkall requested review from whiskyboy and leafy-lee September 21, 2024 09:19

leafy-lee approved these changes Sep 21, 2024

View reviewed changes

thinkall merged commit 49ba962 into main Sep 21, 2024
16 checks passed

thinkall deleted the fix_1355 branch September 21, 2024 12:04