[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: PNG File Screenshot 2019-08-12 at 07.37.44.png     PNG File TXTREC-43-1.png     PNG File TXTREC-43-2.png     PNG File TXTREC-43-3.png     PNG File TXTREC-43-4.png    
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!

Generated at Mon Feb 12 11:04:54 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.