[TXTREC-43] Investigate why semicolon is needed after each text in info.magnolia.ai.text.DefaultTextClassificationService#aggregateTexts(java.util.List<java.lang.String>, javax.jcr.Node) Created: 25/Jul/19 Updated: 19/Aug/19 Resolved: 14/Aug/19 |
|
| Status: | Closed |
| Project: | Text Classification |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Evzen Fochr | Assignee: | Le Hai Thanh |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 3h | ||
| Time Spent: | 6.5h | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoR: |
Empty
|
| Date of First Response: | |
| Epic Link: | Txt Classification integration |
| Sprint: | Add-Ons 18 |
| Story Points: | 3 |
| Description |
|
If it is amazon specific adding it should be moved from info.magnolia.ai.text.DefaultTextClassificationService#appendProperties to info.magnolia.ai.text.amazon.AmazonTextClassifier#classify(java.util.Collection<java.lang.String>) test should be added for it |
| Comments |
| Comment by Le Hai Thanh [ 12/Aug/19 ] |
|
ilgun I totally agree. Let me adjust that logic. Thank you! |