Cancel does NOT work when Perol() called multiple times

[clach04]

After reviewing main() and updating the sample code (from the readme), I now have:

#!/usr/bin/env python
# -*- coding: utf-8 -*-

from percol import Percol
from percol.actions import no_output


def main(candidates):
    with Percol(
            actions=[no_output],
            descriptors={'stdin': None, 'stdout': None, 'stderr': None},
            candidates=iter(candidates)) as p:
        exit_code = p.loop()
    if exit_code == 0:
        results = p.model_candidate.get_selected_results_with_index()
        return [r[0] for r in results]
    else:
        return []

if __name__ == "__main__":
    candidates = ['foo', 'bar', 'baz']
    f = open('sample_list.txt')
    for line in f:
        candidates.append(line)
    f.close()
    print('will prompt 3 times')
    results = main(candidates)
    print("You picked: {!r}".format(results))
    results = main(candidates)
    print("You picked: {!r}".format(results))
    results = main(candidates)
    print("You picked: {!r}".format(results))

This works fine if you select something.

But cancelling using default CTRL-C requires 2 presses of ctrl-c.
Which then results in only displaying once and then each call immediately returns with nothing resulting in nothing 😿

I tried with both:

    exit_code = p.cancel_with_exit_code()  # straight up fails first time
    exit_code = p.finish_with_exit_code(1)  # straight up fails first time

and returns none immediately.

Potentially related:

• how to bind escape key? #56
• How to select nothing? #79

[clach04]

EDIT ignore this comment :-)

This may be a bug in my modified library sample, i.e. user error 🥹 . Using original StringIO code seems to work fine:

from cStringIO import StringIO
from percol import Percol
from percol.actions import no_output

def main(candidates):
    si, so, se = StringIO(), StringIO(), StringIO()
    with Percol(
            actions=[no_output],
            descriptors={'stdin': si, 'stdout': so, 'stderr': se},
            candidates=iter(candidates)) as p:
        p.loop()  # this is here twice and is a mistake
        exit_code = p.loop()
        #exit_code = p.cancel_with_exit_code()  # straight up fails first time
        #exit_code = p.finish_with_exit_code(1)  # straight up fails first time
    if exit_code == 0:
        results = p.model_candidate.get_selected_results_with_index()
        return [r[0] for r in results]
    else:
        return []

if __name__ == "__main__":
    candidates = ['foo', 'bar', 'baz']
    results = main(candidates)
    print("You picked: {!r}".format(results))
    results = main(candidates)
    print("You picked: {!r}".format(results))
    results = main(candidates)
    print("You picked: {!r}".format(results))

[clach04]

I spoke to soon, my modified stdio code had multiple loop() calls, so similar issue:

from cStringIO import StringIO
from percol import Percol
from percol.actions import no_output

def main(candidates):
    si, so, se = StringIO(), StringIO(), StringIO()
    with Percol(
            actions=[no_output],
            descriptors={'stdin': si, 'stdout': so, 'stderr': se},
            candidates=iter(candidates)) as p:
        #p.loop()  # accidentally left this in :-(
        exit_code = p.loop()
        #exit_code = p.cancel_with_exit_code()  # straight up fails first time
        #exit_code = p.finish_with_exit_code(1)  # straight up fails first time
    if exit_code == 0:
        results = p.model_candidate.get_selected_results_with_index()
        return [r[0] for r in results]
    else:
        return []

if __name__ == "__main__":
    candidates = ['foo', 'bar', 'baz']
    results = main(candidates)
    print("You picked: {!r}".format(results))
    results = main(candidates)
    print("You picked: {!r}".format(results))
    results = main(candidates)
    print("You picked: {!r}".format(results))