From 3691e669ae77fd3d0445d4cbe9f3308e5eb9fb85 Mon Sep 17 00:00:00 2001 From: borenet Date: Tue, 30 Jun 2015 10:24:03 -0700 Subject: Add wrapper script for computing n-grams from SKPs on Cluster Telemetry Add --n argument BUG=skia:3840 Review URL: https://codereview.chromium.org/1216073007 --- tools/lua/find_ngrams_on_ct | 61 +++++++++++++++++++++++++++++++++++++++++++++ tools/lua/ngrams.lua | 2 ++ 2 files changed, 63 insertions(+) create mode 100755 tools/lua/find_ngrams_on_ct (limited to 'tools/lua') diff --git a/tools/lua/find_ngrams_on_ct b/tools/lua/find_ngrams_on_ct new file mode 100755 index 0000000000..be3e4c07f1 --- /dev/null +++ b/tools/lua/find_ngrams_on_ct @@ -0,0 +1,61 @@ +#!/usr/bin/env python + + +"""Run Cluster Telemetry to compute n-grams from SKPs.""" + + +import argparse +import os +import re +import subprocess +import sys +import tempfile + + +NGRAMS_LUA_SUBSTITUTION_STR = '-- CHANGEME\nlocal n = \d+\n-- CHANGEME' +SCRIPT_DIR = os.path.dirname(os.path.realpath(__file__)) + + +def main(): + # Parse arguments. + parser = argparse.ArgumentParser( + description='Run Cluster Telemetry to compute n-grams from SKPs.') + parser.add_argument('--n', help='Compute n-grams with this integer as N', + required=True) + args, extra_args = parser.parse_known_args() + + # Read the n-gram Lua script. + script_path = os.path.join(SCRIPT_DIR, 'ngrams.lua') + with open(script_path) as f: + script_contents = f.read() + + # Replace the default value of n with the value specified by the user. + new_contents, subd = re.subn(NGRAMS_LUA_SUBSTITUTION_STR, + 'local n = %s' % args.n, script_contents, 1) + if subd != 1: + raise Exception('Unable to replace N in %s; expected to find:\n%s' % ( + script_path, sub)) + + # Write the new script contents to a temporary file. + tmp_script = tempfile.NamedTemporaryFile(delete=False) + try: + tmp_script.write(new_contents) + tmp_script.close() + + # Run trigger_ct_lua with the new script, forwarding the rest of the + # passed-in arguments to the script. + trigger_ct_lua = os.path.join(SCRIPT_DIR, 'trigger_ct_lua') + cmd = ['python', trigger_ct_lua, + '--script', tmp_script.name, + '--aggregator', os.path.join(SCRIPT_DIR, 'ngrams_aggregate.lua')] + cmd.extend(extra_args) + try: + subprocess.check_call(cmd) + except subprocess.CalledProcessError: + exit(1) + finally: + os.remove(tmp_script.name) + + +if __name__ == '__main__': + main() diff --git a/tools/lua/ngrams.lua b/tools/lua/ngrams.lua index ddbbc9b2db..fee6b902bf 100644 --- a/tools/lua/ngrams.lua +++ b/tools/lua/ngrams.lua @@ -11,7 +11,9 @@ -- box on the same page. -- Change n as desired. +-- CHANGEME local n = 3 +-- CHANGEME -- This algorithm uses a list-of-lists for each SKP. For API call, append a -- list containing just the verb to the master list. Then, backtrack over the -- cgit v1.2.3