Flow function `retry` as non-decorator variant about funcy HOT 2 CLOSED

Given that decorator syntax is just a sugar one might go pre Python 2.4 way:

decorated = decorator(...)(func)

A more specific example can be found in this post (search for "retry"). Or in you case:

def load_file(path: str, retry_tries=None, retry_interval=0.075, unlink=False) -> t.IO[bytes]:
    """..."""
    if unlink:
        try:
            os.unlink(path)
        except:
            pass

    _open = open
    if retry_tries:
        logger.info(f"Retry loading file {path} for {retry_tries} times")
        _open = funcy.retry(tries=int(retry_tries), timeout=float(retry_interval))(open)
    return _open(path, "rb")

Not sure what non-decorator variant would help here, if it is something like:

_open = funcy.retry(open, tries=int(retry_tries), timeout=float(retry_interval))

Then it doesn't fill worth it for me. Actually being even a bit hurtful by providing an extra way to do the same with a non-significant consiseness gain. If you mean some sort of composition with partial for each of funcy's decorator then it also doesn't feel worth it - I would prefer things stay decomposed.

To the last on there is a tangentially related issue, the justification is the same here.

from funcy.

Dear Alexander,

thank you for sharing your excellent article Abstracting Control Flow.

Not sure what non-decorator variant would help here [...]. It doesn't fill worth it for me. Actually being even a bit hurtful by providing an extra way to do the same with a non-significant consiseness gain. If you mean some sort of composition with partial for each of funcy's decorator then it also doesn't feel worth it - I would prefer things stay decomposed.

I can completely follow your point here. Thanks a stack for your swift response.

Also, thank you very much for providing a more readable/structured variant of our load_file() function. I love it!

With kind regards,
Andreas.

from funcy.