Mirror/MarginaliaSearch
1
0
Fork 0
mirror of https://github.com/MarginaliaSearch/MarginaliaSearch.git synced 2025-02-23 21:18:58 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

(index) Remove position overlap check, coherences will do the work instead

Browse Source
This commit is contained in:
Viktor Lofgren 2024-04-17 14:20:01 +02:00
parent 9d008d1d6f
commit dac948973d
1 changed files with 1 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
code/index/java/nu/marginalia/index/results/IndexResultValuationContext.java
View File

@ -97,13 +97,12 @@ public class IndexResultValuationContext {
boolean allSynthetic = !CompiledQueryAggregates.booleanAggregate(wordMetasQuery, WordFlags.Synthetic::isAbsent);
int flagsCount = CompiledQueryAggregates.intMaxMinAggregate(wordMetasQuery, wordMeta -> Long.bitCount(wordMeta & flagsFilterMask));
int positionsCount = CompiledQueryAggregates.intMaxMinAggregate(wordMetasQuery, wordMeta -> Long.bitCount(WordMetadata.decodePositions(wordMeta)));
boolean noneOverlap = wordMetasQuery.root.visit(new PositionOverlapOperator(wordMetasQuery.data)) != 0;
if (!meetsQueryStrategyRequirements(wordMetasQuery, queryParams.queryStrategy())) {
return null;
}
if (flagsCount == 0 && !allSynthetic && (positionsCount == 0 || noneOverlap))
if (flagsCount == 0 && !allSynthetic && positionsCount == 0)
return null;
double score = searchResultValuator.calculateSearchResultValue(
@ -167,42 +166,3 @@ public class IndexResultValuationContext {
}
}
class PositionOverlapOperator implements CqExpression.LongVisitor {
private final CqDataLong wordMetaData;
PositionOverlapOperator(CqDataLong wordMetaData) {
this.wordMetaData = wordMetaData;
}
@Override
public long onAnd(List<? extends CqExpression> parts) {
long positions = ~0;
long flags = 0;
for (var part : parts) {
long pv = part.visit(this);
flags |= pv;
positions &= pv;
}
return positions | (flags & WordMetadata.FLAGS_MASK);
}
@Override
public long onOr(List<? extends CqExpression> parts) {
long ret = 0;
for (var part : parts) {
ret |= part.visit(this);
}
return ret;
}
@Override
public long onLeaf(int idx) {
return wordMetaData.get(idx);
}
}