)]}'
{"monitoring/static/monitoring/js/controllers.js":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"eeee02cfc7fd4b747524a935f325535f4342d554","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        $scope.updateSporadicInfo \u003d function() {"},{"line_number":208,"context_line":"            for (i \u003d 0; i \u003c $scope.metrics.length; i++) {"},{"line_number":209,"context_line":"                metric \u003d $scope.metrics[i];"},{"line_number":210,"context_line":"                if (metric.name \u003d\u003d\u003d $scope.currentMetric) {"},{"line_number":211,"context_line":"                    $scope.currentSporadic \u003d metric.sporadic;"},{"line_number":212,"context_line":"                    break;"},{"line_number":213,"context_line":"                }"}],"source_content_type":"text/javascript","patch_set":1,"id":"dada55a8_99b995ce","line":210,"updated":"2016-07-19 11:55:34.000000000","message":"So the name of metric means that it is sporadic ?\nLooks odd, would you explain this part of the code a bit ?","commit_id":"03eaf9b94dd23001d782635b1d2c1e1140ff5adb"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"2d35ec3dbf41ca00b929c90b2316c675f5c1bc4d","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        $scope.updateSporadicInfo \u003d function() {"},{"line_number":208,"context_line":"            for (i \u003d 0; i \u003c $scope.metrics.length; i++) {"},{"line_number":209,"context_line":"                metric \u003d $scope.metrics[i];"},{"line_number":210,"context_line":"                if (metric.name \u003d\u003d\u003d $scope.currentMetric) {"},{"line_number":211,"context_line":"                    $scope.currentSporadic \u003d metric.sporadic;"},{"line_number":212,"context_line":"                    break;"},{"line_number":213,"context_line":"                }"}],"source_content_type":"text/javascript","patch_set":1,"id":"dada55a8_19545bfd","line":210,"in_reply_to":"dada55a8_39f336d8","updated":"2016-07-20 04:38:04.000000000","message":"Yes, it is :).\nThough, I had little different idea of implementing that.\nI wanted to make the box where you select metric rich, that said to display an icon next to metric name (on right/left) with some icon. There is plenty of icons @ font-awesome ;-).\n\nWhat do you think of such idea ?","commit_id":"03eaf9b94dd23001d782635b1d2c1e1140ff5adb"},{"author":{"_account_id":15359,"name":"Erickson Filipe Guedes dos Santos","email":"erickson@lsd.ufcg.edu.br","username":"ericksonfilipe"},"change_message_id":"789b5f853e7c0ab83d28f11d7758454d22eb0b7f","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        $scope.updateSporadicInfo \u003d function() {"},{"line_number":208,"context_line":"            for (i \u003d 0; i \u003c $scope.metrics.length; i++) {"},{"line_number":209,"context_line":"                metric \u003d $scope.metrics[i];"},{"line_number":210,"context_line":"                if (metric.name \u003d\u003d\u003d $scope.currentMetric) {"},{"line_number":211,"context_line":"                    $scope.currentSporadic \u003d metric.sporadic;"},{"line_number":212,"context_line":"                    break;"},{"line_number":213,"context_line":"                }"}],"source_content_type":"text/javascript","patch_set":1,"id":"dada55a8_39f336d8","line":210,"in_reply_to":"dada55a8_99b995ce","updated":"2016-07-19 16:53:45.000000000","message":"No, actually, currentMetric only holds the name of the current selected metric in the ui. That said, in order to update the sporadic field in the modal, we need to know if the selected metric ($scope.currentMetric) is sporadic.\n\nTo get the information if the currentMetric  is sporadic, we are iterating over the metric list (which holds the sporadic information for each metric).\n\nI Hope it\u0027s clear now.","commit_id":"03eaf9b94dd23001d782635b1d2c1e1140ff5adb"}]}