Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(132)

Side by Side Diff: appengine/findit/crash/scorers/top_frame_index.py

Issue 1861373003: [Findit] Initial code of findit for crash. Add scorers to apply heuristic rules. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Fix nits. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
5 """TopFrameIndex scorer applies to all Result objects.
6
7 It represents a heuristic rule:
8 The less the top frame index (this result changed) is, the higher score.
9 """
10
11 from crash.scorers.scorer import Scorer
12
13 _MAX_TOP_N_FRAMES = 7
stgao 2016/04/21 17:31:54 This should go into the config saved in datastore.
Sharu Jiang 2016/04/21 22:38:50 Done.
14 _INFINITY = 1000
15
16
17 class TopFrameIndex(Scorer):
18
19 def __init__(self, max_top_n=_MAX_TOP_N_FRAMES):
20 self.max_top_n = max_top_n
21
22 def GetMetric(self, result):
23 if not result.file_to_stack_infos:
24 return None
25
26 top_frame_index = _INFINITY
27 for _, stack_infos in result.file_to_stack_infos.iteritems():
28 for frame, _ in stack_infos:
29 top_frame_index = min(top_frame_index, frame.index)
30
31 return top_frame_index
32
33 def Score(self, top_frame_index):
34 # TODO(katesonia): experiment the model and parameters later.
35 if top_frame_index < self.max_top_n:
36 return 1 - top_frame_index / float(self.max_top_n)
37
38 return 0
39
40 def Reason(self, top_frame_index, score):
41 if score == 0:
42 return ''
43
44 return 'Top frame changed is frame #%d' % top_frame_index
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698