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

Handling of words absence in vocabulary #7154

Merged
merged 6 commits into from Feb 12, 2019

Conversation

Projects
None yet
2 participants
@alexanderst
Copy link
Contributor

commented Feb 12, 2019

What changes were proposed in this pull request?

Better handling of case when labels are absent in vocabulary for different configurations.

How was this patch tested?

Unit testing.

Quick checklist

The following checklist helps ensure your PR is complete:

  • Reviewed the Contributing Guidelines and followed the steps within.
  • Created tests for any significant new code additions.
  • Relevant tests for your changes are passing.
  • Ran mvn formatter:format (see formatter instructions for targeting your specific files).

@raver119 raver119 merged commit 2ffc5b3 into master Feb 12, 2019

0 of 2 checks passed

codeclimate Code Climate encountered an error attempting to analyze this pull request.
Details
continuous-integration/jenkins/pr-head This commit is being built
Details

@raver119 raver119 deleted the asto_GracefullyUnknown branch Feb 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.