Skip to content

Add time series regression to LightGBMRegressor supported problem types#2793

Merged
freddyaboulton merged 2 commits intomainfrom
2760-add-lightgbm-to-time-series
Sep 17, 2021
Merged

Add time series regression to LightGBMRegressor supported problem types#2793
freddyaboulton merged 2 commits intomainfrom
2760-add-lightgbm-to-time-series

Conversation

@freddyaboulton
Copy link
Contributor

Pull Request Description

Fixes #2760

Performance tests here: https://alteryx.atlassian.net/wiki/spaces/PS/pages/1046872126/Adding+LightGBM+regressor+to+time+series+estimators+used+in+AutoMLSearch


After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

@freddyaboulton freddyaboulton changed the title Add ts regression to lgbm Add time series regression to LightGBMRegressor supported problem types Sep 16, 2021
@codecov
Copy link

codecov bot commented Sep 16, 2021

Codecov Report

Merging #2793 (18a0689) into main (916f797) will increase coverage by 0.1%.
The diff coverage is 100.0%.

❗ Current head 18a0689 differs from pull request most recent head 38cb89c. Consider uploading reports for the commit 38cb89c to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##            main   #2793     +/-   ##
=======================================
+ Coverage   99.8%   99.8%   +0.1%     
=======================================
  Files        297     298      +1     
  Lines      27718   27731     +13     
=======================================
+ Hits       27650   27663     +13     
  Misses        68      68             
Impacted Files Coverage Δ
...onents/estimators/regressors/lightgbm_regressor.py 100.0% <100.0%> (ø)
...valml/tests/component_tests/test_lgbm_regressor.py 100.0% <100.0%> (ø)
evalml/data_checks/__init__.py 100.0% <0.0%> (ø)
evalml/tests/data_checks_tests/test_data_checks.py 100.0% <0.0%> (ø)
...valml/tests/pipeline_tests/test_component_graph.py 100.0% <0.0%> (ø)
evalml/data_checks/utils.py 100.0% <0.0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2bc8654...38cb89c. Read the comment docs.

Copy link
Collaborator

@jeremyliweishih jeremyliweishih left a comment

Choose a reason for hiding this comment

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

👏

Copy link
Contributor

@angela97lin angela97lin left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@freddyaboulton freddyaboulton force-pushed the 2760-add-lightgbm-to-time-series branch 2 times, most recently from bd85284 to 173c9df Compare September 17, 2021 15:12
@freddyaboulton freddyaboulton force-pushed the 2760-add-lightgbm-to-time-series branch from 173c9df to 38cb89c Compare September 17, 2021 16:13
@freddyaboulton freddyaboulton merged commit 66e6afe into main Sep 17, 2021
@freddyaboulton freddyaboulton deleted the 2760-add-lightgbm-to-time-series branch September 17, 2021 16:46
@chukarsten chukarsten mentioned this pull request Oct 1, 2021
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.

Add time series regression to LightGBMRegressor's supported problem types

4 participants