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

additional regressor for prophet does not work #63

Open
sykrn opened this issue Jul 31, 2023 · 5 comments
Open

additional regressor for prophet does not work #63

sykrn opened this issue Jul 31, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@sykrn
Copy link

sykrn commented Jul 31, 2023

The problem why additional regressor for prophet does not work due to these lines.
the regr object overrides the previous model with additional regressors.

    model = Prophet(**kwargs)
    for x in Xvars:
        model.add_regressor(x)
    regr = Prophet(**kwargs)
    regr.fit(X)
    fcst = regr.predict(p)

https://github.com/mikekeith52/scalecast/blob/1a8301784fcd60b07bb963896c83ca1ccdf244cd/src/scalecast/Forecaster.py#L561C1-L566C31

@mikekeith52 mikekeith52 self-assigned this Jul 31, 2023
@mikekeith52 mikekeith52 added the bug Something isn't working label Jul 31, 2023
@mikekeith52
Copy link
Owner

Oof. That's a pretty egregious mistake on my part. I will have that fixed very soon, in the next release.

@sykrn
Copy link
Author

sykrn commented Jul 31, 2023

Thanks @mikekeith52, waiting for the next release. :)

mikekeith52 added a commit that referenced this issue Jul 31, 2023
- Fixed an issue where Prophet was not adding additional regressors correctly (#63).
@mikekeith52 mikekeith52 mentioned this issue Jul 31, 2023
@mikekeith52
Copy link
Owner

Can I get you to review pull request #64? I confirmed that the code works but want to get your eyes on it to make sure there are no additional problems than what I was able to change.

@mikekeith52
Copy link
Owner

You should be able to upgrade to 0.18.11: pip install --upgrade scalecast

@mikekeith52
Copy link
Owner

Can we close this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants