[MGNLPER-90] LRU Policy instead of FIFO Created: 12/Feb/19 Updated: 14/Jun/19 Resolved: 26/Feb/19 |
|
| Status: | Closed |
| Project: | Periscope |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.1 |
| Type: | Story | Priority: | Neutral |
| Reporter: | Simon Lutz | Assignee: | Cedric Reichenbach |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Template: |
|
||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||
| Release notes required: |
Yes
|
||||||||||||
| Documentation update required: |
Yes
|
||||||||||||
| Date of First Response: | |||||||||||||
| Epic Link: | Search relevance | ||||||||||||
| Sprint: | Foundation 4, Foundation 5 | ||||||||||||
| Story Points: | 5 | ||||||||||||
| Description |
|
Currently we drop results from the output label according to a FIFO-policy. Also see comments on the linked ticket. |
| Comments |
| Comment by Cedric Reichenbach [ 15/Feb/19 ] |
|
Documentation updates will only be necessary if the NN result ranker "memory" is actually subject to documentation (not sure if it's planned, hence set the tick to be on the safe side). |
| Comment by Christoph Meier [ 21/May/19 ] |
|
fgrilli, cc slutz ... I guess this has been implemented, right? Currently docu claims:
That should be enough, right? |
| Comment by Simon Lutz [ 21/May/19 ] |
|
Yes, this is implemented and to me the documentation is sufficient as it stands now. |