human
I wrote the below code

    def fix(p):
        if ((not p) or (str(p).lower() == 'none')):
            return ''
        else:
            return UNTRANS(str(p))
    values = [1, name, pp, cat, script, priority, None]
    script_path = make_script_path(cfg.pre_script())
    if script_path:
        command = [script_path, name, fix(pp), fix(cat), fix(script), fix(priority), str(size), '    '.join(groups)]
        command.extend(analyse_show(name))
        try:
            (stup, need_shell, command, creationflags) = build_command(command)
            env = create_env()
            logging.info('Running    pre-queue    script    %s', command)
            p = Popen(command, shell=need_shell, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, startupinfo=stup, env=env, creationflags=creationflags)
        except:
            logging.debug('Failed    script    %s,    Traceback:    ', script_path, exc_info=True)
            return values
        output = p.stdout.read()
        ret = p.wait()
        logging.info('Pre-queue    script    returns    %s    and    output=\n%s', ret, output)
        if (ret == 0):
            n = 0
            for line in output.split('\n'):
                line = line.strip('\r\n    \'"')
                if ((n < len(values)) and line):
                    values[n] = TRANS(line)
                n += 1
        accept = int_conv(values[0])
        if (accept < 1):
            logging.info('Pre-Q    refuses    %s', name)
        elif (accept == 2):
            logging.info('Pre-Q    accepts&fails    %s', name)
        else:
            logging.info('Pre-Q    accepts    %s', name)
    return values

What's a good function header?

Here's a good function header?
assistant
#Here's a good function header.

def pre_queue(name, pp, cat, script, priority, size, groups):