-
Notifications
You must be signed in to change notification settings - Fork 13
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
RD-1983 add support for topics and keyphrases #25
RD-1983 add support for topics and keyphrases #25
Conversation
protected Double confidence; | ||
protected String conceptId; | ||
|
||
public Builder(String concept, Double confidence, String conceptId) { |
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.
Seems you want a default constructor (or minimum requirement only) also, or you can skip other methods..?
private final Double confidence; | ||
private final String conceptId; | ||
|
||
public Topic() throws Exception { |
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.
whoops, this wasn't meant to make it in. Will remove
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.
:)
@JsonCreator | ||
ConceptMixin(@JsonProperty("concept") String concept, | ||
@JsonProperty("salience") Double salience, | ||
@JsonProperty("conceptId") String topicId, |
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 this intentionally different between json and java?
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.
nope, just missed in the big concept <-> topic switch. pushed a fix
No description provided.