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.

Sign up for GitHub

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

Jump to bottom

Forecasting model parallelization #511

Merged

codeloop merged 8 commits into feature/forecast_v2 from feature/model_parallel

Jan 9, 2024

Member

codeloop commented Jan 3, 2024

Forecasting model parallelization ODSC-50382

make the model parallel using joblib
allow object parallel access,
some restructuring and formatting


          make the model parallel, allow object parallel access, some restructu…

d9bab09

…ring and formatting

oracle-contributor-agreement bot added the OCA Verified label

github-actions bot commented Jan 3, 2024

📌 Cov diff with main:

📌 Overall coverage:


          add failed model to error dict and write it to a file in case of failure

2c53fa2

github-actions bot commented Jan 3, 2024

📌 Cov diff with main:

📌 Overall coverage:

codeloop added 2 commits

January 4, 2024 17:40


          add the joblib parallelisation for the prophet, update the automlx pa…

4394f67

…ram to inter-model and intra models parallel process


          revert changes

177fa40

github-actions bot commented Jan 4, 2024

📌 Cov diff with main:

No success to gather report. 😿

📌 Overall coverage:

No success to gather report. 😿

github-actions bot commented Jan 4, 2024

📌 Cov diff with main:

📌 Overall coverage:


          make neural prophet parallel mdoel training

5242f1d

codeloop requested review from mrDzurb, prasankh, ahosler and govarsha

January 5, 2024 06:38


          removed completed todo

577cb23

github-actions bot commented Jan 5, 2024

📌 Cov diff with main:

No success to gather report. 😿

📌 Overall coverage:

No success to gather report. 😿


          removed commented code

9e65a16

github-actions bot commented Jan 5, 2024

📌 Cov diff with main:

No success to gather report. 😿

📌 Overall coverage:

No success to gather report. 😿

github-actions bot commented Jan 5, 2024

📌 Cov diff with main:

📌 Overall coverage:

ahosler reviewed

View reviewed changes

Member

ahosler left a comment

Works well locally

ads/opctl/operator/lowcode/forecast/model/arima.py Outdated Show resolved Hide resolved

ads/opctl/operator/lowcode/forecast/operator_config.py Outdated Show resolved Hide resolved

ahosler reviewed

View reviewed changes

ads/opctl/operator/lowcode/forecast/model/neuralprophet.py Outdated Show resolved Hide resolved

ahosler approved these changes

View reviewed changes

Member

ahosler left a comment •

edited

Loading

This looks awesome! Great work, this will be a huge efficiency gain for users.


          resolve pr comments:

bccdd30

ahosler approved these changes

View reviewed changes

Member

ahosler left a comment

This looks great! I tested with each framework and noticed very meaningful speedups.

codeloop merged commit aecf4d6 into feature/forecast_v2

9 of 13 checks passed

github-actions bot commented Jan 9, 2024

📌 Cov diff with main:

📌 Overall coverage:

ahosler deleted the feature/model_parallel branch

January 9, 2024 08:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels