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
refactor: migrate lda finish_example #4413
Conversation
This reverts commit 9c25315.
ld->v.resize(all.lda * ld->minibatch); | ||
|
||
ld->decay_levels.push_back(0.f); | ||
|
||
all.example_parser->lbl_parser = VW::no_label_parser_global; | ||
|
||
// If minibatch is > 1, then the predict function does not actually produce predictions. |
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.
We should probably fix this before we merge? Or at the very least get a John opinion on if this is ok for the time being
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.
Its existed this way for a long time (probably since the beginning), we just correctly report the type now. That is, that it produces no prediction.
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.
Discussed offline, its fine since it is maintaining previous behavior
Performance test shows no noticeable change in performance.