MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-09-27 10:03:11 +07:00
Code Issues Packages Projects Releases Wiki Activity

[stats-collector] move checks on features logging in experiment to ExperimentInfo

Browse Source 
(cherry picked from commit 31cfee81bca3ce23cdeef1c9625a2670cc13be2c)

IJ-CR-26550

GitOrigin-RevId: 66caba1ab2748d8489120e155d184fe5a9922551
This commit is contained in:
Alexey Kalina
2022-07-27 15:55:04 +00:00
committed by intellij-monorepo-bot
parent dc54e172ce
commit 91c1f6cdec
2 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/completion-ml-ranking/src/com/intellij/completion/ml/experiment/ExperimentInfo.kt
+10 -1
View File
@@ -1,9 +1,18 @@
// Copyright 2000-2022 JetBrains s.r.o. Use of this source code is governed by the Apache 2.0 license that can be found in the LICENSE file.
package com.intellij.completion.ml.experiment
import com.intellij.completion.ml.common.CurrentProjectInfo
import com.intellij.openapi.project.Project
data class ExperimentInfo(val inExperiment: Boolean,
val version: Int,
val shouldRank: Boolean = false,
val shouldShowArrows: Boolean = false,
val shouldCalculateFeatures: Boolean = false,
val shouldLogElementFeatures: Boolean = false)
val shouldLogElementFeatures: Boolean = false) {
fun shouldLogSessions(project: Project): Boolean = inExperiment || CurrentProjectInfo.getInstance(project).isIdeaProject
fun shouldLogElementFeatures(project: Project): Boolean =
!inExperiment || shouldCalculateFeatures && (shouldLogElementFeatures || CurrentProjectInfo.getInstance(project).isIdeaProject)
}
plugins/stats-collector/src/com/intellij/stats/completion/tracker/CompletionLoggerInitializer.kt
+3 -8
View File
@@ -2,7 +2,6 @@
package com.intellij.stats.completion.tracker
import com.intellij.codeInsight.lookup.impl.LookupImpl
import com.intellij.completion.ml.common.CurrentProjectInfo
import com.intellij.completion.ml.experiment.ExperimentInfo
import com.intellij.completion.ml.experiment.ExperimentStatus
import com.intellij.completion.ml.storage.MutableLookupStorage
@@ -78,17 +77,13 @@ class CompletionLoggerInitializer : LookupTracker() {
return LoggerPerformanceTracker(actionsTracker, storage.performanceTracker)
}
private fun shouldLogElementFeatures(language: Language, project: Project): Boolean {
val experimentInfo = ExperimentStatus.getInstance().forLanguage(language)
return !experimentInfo.inExperiment ||
(experimentInfo.shouldCalculateFeatures && (experimentInfo.shouldLogElementFeatures ||
CurrentProjectInfo.getInstance(project).isIdeaProject))
}
private fun shouldLogElementFeatures(language: Language, project: Project): Boolean =
ExperimentStatus.getInstance().forLanguage(language).shouldLogElementFeatures(project)
private fun sessionShouldBeLogged(experimentInfo: ExperimentInfo, language: Language, project: Project): Boolean {
if (CompletionStatsPolicy.isStatsLogDisabled(language) || !getPluginInfo(language::class.java).isSafeToReport()) return false
val application = ApplicationManager.getApplication()
if (application.isUnitTestMode || experimentInfo.inExperiment || CurrentProjectInfo.getInstance(project).isIdeaProject) return true
if (application.isUnitTestMode || experimentInfo.shouldLogSessions(project)) return true
if (!isCompletionLogsSendAllowed()) {
return false