Commit 4b2b9421 authored by tby's avatar tby Committed by Commit Bot

[Files ranking] Remove VLOG statements.

We've got some very spammy vlogging to debug an issue we've fixed,
let's remove it.

Bug: 959679
Change-Id: Iff5bbbe4f3745b3d44059e591c137276b141c8d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2114773Reviewed-by: default avatarThanh Nguyen <thanhdng@chromium.org>
Commit-Queue: Tony Yeoman <tby@chromium.org>
Cr-Commit-Position: refs/heads/master@{#752379}
parent 43ba11ed
...@@ -516,15 +516,6 @@ void SearchResultRanker::Train(const AppLaunchData& app_launch_data) { ...@@ -516,15 +516,6 @@ void SearchResultRanker::Train(const AppLaunchData& app_launch_data) {
if (model == Model::MIXED_TYPES && app_launch_data.query.empty()) { if (model == Model::MIXED_TYPES && app_launch_data.query.empty()) {
LogZeroStateLaunchType(app_launch_data.ranking_item_type); LogZeroStateLaunchType(app_launch_data.ranking_item_type);
if (zero_state_group_ranker_) {
std::vector<std::string> weights;
for (const auto& pair : *zero_state_group_ranker_->GetTargetData())
weights.push_back(base::StrCat(
{pair.first, ":", base::NumberToString(pair.second.last_score)}));
VLOG(1) << "Zero state files model weights: ["
<< base::JoinString(weights, ", ") << "]";
}
} }
LogChipUsageMetrics(app_launch_data); LogChipUsageMetrics(app_launch_data);
...@@ -599,26 +590,12 @@ void SearchResultRanker::OverrideZeroStateResults( ...@@ -599,26 +590,12 @@ void SearchResultRanker::OverrideZeroStateResults(
if (candidate_override_index == -1) if (candidate_override_index == -1)
continue; continue;
// TODO(crbug.com/1011221): Remove once the bug re. zero-state drive files
// not being shown is resolved.
VLOG(1) << "Zero state files override: newtype=" << static_cast<int>(group)
<< " newpos=" << next_modifiable_index;
// Override the result at |next_modifiable_index| with // Override the result at |next_modifiable_index| with
// |candidate_override_index| by swapping their scores. // |candidate_override_index| by swapping their scores.
std::swap(result_ptrs[candidate_override_index]->score, std::swap(result_ptrs[candidate_override_index]->score,
result_ptrs[next_modifiable_index]->score); result_ptrs[next_modifiable_index]->score);
--next_modifiable_index; --next_modifiable_index;
} }
// TODO(crbug.com/1011221): Remove once the bug re. zero-state drive files not
// being shown is resolved.
VLOG(1) << "Zero state files setting result scores";
for (const auto* result : result_ptrs) {
VLOG(1) << "Zero state files result score: type="
<< static_cast<int>(
RankingItemTypeFromSearchResult(*(result->result)))
<< " score=" << result->score;
}
} }
void SearchResultRanker::OnFilesOpened( void SearchResultRanker::OnFilesOpened(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment