Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Optimisation of historical forecast for regression models #1885
Optimisation of historical forecast for regression models #1885
Changes from 49 commits
b7a2040
9b43b51
c37193e
d47ff70
4ed475a
7231bab
b261cde
3b24330
a319d99
76263f2
c4896fa
e64e962
147e1f8
307c0e9
b8a1e94
6d8ea99
2fd1971
9af8539
9a372bd
9cd0a8f
b2a5185
30d5b7b
685417d
c309a44
c5f0236
a544128
9e479b6
935a426
c8e74e4
f5d936e
76534a1
dff4652
84748ad
97aa91e
78b97da
09959a5
be7dd88
2a45588
7f4952c
b4b21a0
ce3b00f
29e7211
2ddb537
cd0078d
e277d80
572721b
0edd779
8c98d48
102178d
ce508ea
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in the current implemenetation models using encoders are not yet optimizable.
Do you think we could add support for this?
Edit: I'm thinking about adding something like a
generate_fit_predict_encodings
which would make this a bit easier. Maybe we could drop optimization support for encoders until then.Edit 2: I added the
generate_fit_predict_encodings
in #1925. Would be cool to merge that one and then add the support for optimization with encodings here as well :)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, I tend to forget about the encoders... Thank you for implementing this, I will adjust this PR as soon as the other one is merged.