)]}'
{"vitrage/entity_graph/consistency/consistency_enforcer.py":[{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"babcaee9c2b60a36ce88329dcdde5c5a2d932cda","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    def _run_evaluator(self, vertices):"},{"line_number":135,"context_line":"        start_time \u003d time.time()"},{"line_number":136,"context_line":"        for vertex in vertices:"},{"line_number":137,"context_line":"            self.evaluator.process_event(None, vertex, True, self.graph)"},{"line_number":138,"context_line":"        LOG.info(\u0027Run Evaluator on %s items - took %s\u0027, str(len(vertices)),"},{"line_number":139,"context_line":"                 str(time.time() - start_time))"},{"line_number":140,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_f36fb3f9","line":137,"range":{"start_line":137,"start_character":61,"end_line":137,"end_character":65},"updated":"2016-08-17 15:43:04.000000000","message":"The evaluator already holds the graph, so let\u0027s not pass the graph again. See my comment on scenario_evaluator","commit_id":"8d83576cb1dd910cc67dc32b077de3fb941d2898"}],"vitrage/evaluator/scenario_evaluator.py":[{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"babcaee9c2b60a36ce88329dcdde5c5a2d932cda","unresolved":false,"context_lines":[{"line_number":69,"context_line":"    def scenario_repo(self, scenario_repo):"},{"line_number":70,"context_line":"        self._scenario_repo \u003d scenario_repo"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def process_event(self, before, current, is_vertex, graph):"},{"line_number":73,"context_line":"        \"\"\"Notification of a change in the entity graph."},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"        :param is_vertex:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_33014b39","line":72,"range":{"start_line":72,"start_character":56,"end_line":72,"end_character":61},"updated":"2016-08-17 15:43:04.000000000","message":"The evaluator already has \u0027entity_graph\u0027, so this is confusing. Maybe you should change it to kwargs and ignore it (as you are doing anyway)","commit_id":"8d83576cb1dd910cc67dc32b077de3fb941d2898"}]}