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
Increase scikit-learn version #1635
Conversation
@@ -178,7 +178,7 @@ def process(self, message, **kwargs): | |||
else: | |||
X = message.get("text_features").reshape(1, -1) | |||
intent_ids, probabilities = self.predict(X) | |||
intents = self.transform_labels_num2str(intent_ids) | |||
intents = self.transform_labels_num2str(intent_ids.T) |
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.
is that correct or should it be np.ravel(intent_ids)
?
Also, after this change, what happens if we run with 0.19, does it still work?
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.
I've included the change, but we flatten this vector in the following line so it's not an issue
Just ran the script with 0.19, it still passes
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.
awesome, thanks 👍
Code Climate has analyzed commit 3338ee2 and detected 0 issues on this pull request. View more on Code Climate. |
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.
please make sure to also fix the version pinning in setup.py
- after that it is good to merge
Flag config parameters passed to agent.train(), remove config params from training params
…aHQ#1635) Bumps [github.com/moby/buildkit](https://github.com/moby/buildkit) from 0.11.2 to 0.11.3. - [Release notes](https://github.com/moby/buildkit/releases) - [Commits](moby/buildkit@v0.11.2...v0.11.3) --- updated-dependencies: - dependency-name: github.com/moby/buildkit dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
scikit-learn==0.20.2
has a niceoutput_dict
feature we can use in our evaluate script, but it was causing errors with our classifierProposed changes:
intent_ids
vectorscikit-learn==0.20.2
Status (please check what you already did):