Skip to content

Commit e3641ce

Browse files
committed
fix review11: refactos
1 parent 6cbd929 commit e3641ce

File tree

4 files changed

+8
-13
lines changed

4 files changed

+8
-13
lines changed

extra/modules/greenbids-real-time-data/src/main/java/org/prebid/server/hooks/modules/greenbids/real/time/data/core/GreenbidsInferenceDataService.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,7 @@ public List<ThrottlingMessage> extractThrottlingMessagesFromBidRequest(BidReques
4545
.map(GreenbidsUserAgent::new)
4646
.orElse(null);
4747

48-
return extractThrottlingMessages(
49-
bidRequest,
50-
userAgent);
48+
return extractThrottlingMessages(bidRequest, userAgent);
5149
}
5250

5351
private List<ThrottlingMessage> extractThrottlingMessages(

extra/modules/greenbids-real-time-data/src/main/java/org/prebid/server/hooks/modules/greenbids/real/time/data/core/GreenbidsInvocationService.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,6 @@ private Ortb2ImpExtResult createOrtb2ImpExt(
115115
final Map<String, Boolean> impBiddersFilterMap = impsBiddersFilterMap.get(imp.getId());
116116
final ExplorationResult explorationResult = ExplorationResult.of(
117117
greenbidsId, impBiddersFilterMap, isExploration);
118-
return Ortb2ImpExtResult.of(
119-
explorationResult, tid);
118+
return Ortb2ImpExtResult.of(explorationResult, tid);
120119
}
121120
}

extra/modules/greenbids-real-time-data/src/main/java/org/prebid/server/hooks/modules/greenbids/real/time/data/core/GreenbidsUserAgent.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -42,19 +42,17 @@ public String getDevice() {
4242
public String getBrowser() {
4343
return Optional.ofNullable(userAgent)
4444
.filter(userAgent -> !"Other".equals(userAgent.family) && StringUtils.isNoneBlank(userAgent.family))
45-
.map(userAgent -> {
46-
final String major = Optional.ofNullable(userAgent.major).orElse(StringUtils.EMPTY);
47-
return "%s %s".formatted(userAgent.family, major).trim();
48-
})
45+
.map(ua -> "%s %s".formatted(ua.family, StringUtils.defaultString(userAgent.major)).trim())
4946
.orElse(StringUtils.EMPTY);
5047
}
5148

5249
private boolean isPC() {
50+
final String osFamily = osFamily();
5351
return Optional.ofNullable(userAgentString)
5452
.map(userAgent -> userAgent.contains("Windows NT")
55-
|| PC_OS_FAMILIES.contains(osFamily())
56-
|| ("Windows".equals(osFamily()) && "ME".equals(osMajor()))
57-
|| ("Mac OS X".equals(osFamily()) && !userAgent.contains("Silk"))
53+
|| PC_OS_FAMILIES.contains(osFamily)
54+
|| ("Windows".equals(osFamily) && "ME".equals(osMajor()))
55+
|| ("Mac OS X".equals(osFamily) && !userAgent.contains("Silk"))
5856
|| (userAgent.contains("Linux") && userAgent.contains("X11")))
5957
.orElse(false);
6058
}

extra/modules/greenbids-real-time-data/src/main/java/org/prebid/server/hooks/modules/greenbids/real/time/data/core/OnnxModelRunner.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ public OnnxModelRunner(byte[] onnxModelBytes) throws OrtException {
1818
}
1919

2020
public OrtSession.Result runModel(String[][] throttlingInferenceRow) throws OrtException {
21-
final OnnxTensor inputTensor = OnnxTensor.createTensor(OrtEnvironment.getEnvironment(), throttlingInferenceRow);
21+
final OnnxTensor inputTensor = OnnxTensor.createTensor(ENVIRONMENT, throttlingInferenceRow);
2222
return session.run(Collections.singletonMap("input", inputTensor));
2323
}
2424
}

0 commit comments

Comments
 (0)