From patchwork Fri May 25 15:58:28 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: leonardo.sandoval.gonzalez@linux.intel.com X-Patchwork-Id: 27502 Received: (qmail 106570 invoked by alias); 25 May 2018 15:58:39 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 106553 invoked by uid 89); 25 May 2018 15:58:38 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-24.4 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_LAZY_DOMAIN_SECURITY autolearn=ham version=3.3.2 spammy= X-HELO: mga17.intel.com X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 From: leonardo.sandoval.gonzalez@linux.intel.com To: libc-alpha@sourceware.org Cc: Leonardo Sandoval Subject: [PATCH 1/3] benchtests: Add --no-diff option to avoid diff calculation Date: Fri, 25 May 2018 10:58:28 -0500 Message-Id: <20180525155830.6864-2-leonardo.sandoval.gonzalez@linux.intel.com> In-Reply-To: <20180525155830.6864-1-leonardo.sandoval.gonzalez@linux.intel.com> References: <20180525155830.6864-1-leonardo.sandoval.gonzalez@linux.intel.com> From: Leonardo Sandoval Having the report with no diff numbers (percentages numbers enclosed within parenthesis) makes column separation cleaner (just spaces), allowing easier interaction with other tools. * benchtests/scripts/compare_string.py: Add --no-diff option to avoid diff calculation. --- ChangeLog | 5 +++++ benchtests/scripts/compare_strings.py | 15 +++++++++------ 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index c7436c4f333..afddd77315a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2018-05-25 Leonardo Sandoval + + * benchtests/scripts/compare_string.py: Add --no-diff option + to avoid diff calculation. + 2018-05-24 Tulio Magno Quites Machado Filho Gabriel F. T. Gomes diff --git a/benchtests/scripts/compare_strings.py b/benchtests/scripts/compare_strings.py index d37442076bf..ceeccea13b6 100755 --- a/benchtests/scripts/compare_strings.py +++ b/benchtests/scripts/compare_strings.py @@ -79,7 +79,7 @@ def draw_graph(f, v, ifuncs, results): pylab.savefig('%s-%s.png' % (f, v), bbox_inches='tight') -def process_results(results, attrs, base_func, graph): +def process_results(results, attrs, base_func, graph, no_diff): """ Process results and print them Args: @@ -107,10 +107,11 @@ def process_results(results, attrs, base_func, graph): graph_res[key] = res['timings'] for t in res['timings']: sys.stdout.write ('%12.2f' % t) - if i != base_index: - base = res['timings'][base_index] - diff = (base - t) * 100 / base - sys.stdout.write (' (%6.2f%%)' % diff) + if not no_diff: + if i != base_index: + base = res['timings'][base_index] + diff = (base - t) * 100 / base + sys.stdout.write (' (%6.2f%%)' % diff) sys.stdout.write('\t') i = i + 1 print('') @@ -132,7 +133,7 @@ def main(args): attrs = args.attributes.split(',') results = parse_file(args.input, args.schema) - process_results(results, attrs, base_func, args.graph) + process_results(results, attrs, base_func, args.graph, args.no_diff) if __name__ == '__main__': @@ -152,6 +153,8 @@ if __name__ == '__main__': help='IFUNC variant to set as baseline.') parser.add_argument('-g', '--graph', action='store_true', help='Generate a graph from results.') + parser.add_argument('--no-diff', action='store_true', + help='Do not include diff on .') args = parser.parse_args() main(args)