resolve all quacc/prefect flow futures to data make prefect happy (#…

[zulissimeta]

…2460)

Summary of Changes

Prefect3 now requires all jobs to be resolved before a flow exits. If you return a job that is still running, this causes warnings and problems.

Also, return futures/states is good in that you can avoid passing around data until it's needed, but annoying in that you may ended up with complex structures that have prefect futures embedded inside them.

This PR introduces a function resolve_futures_to_results modeled on the prefect function resolve_futures_to_states that will recursively crawl python collections and resolve any futures to their final results. This function is called on any results that are returned by a quacc flow so that they are properly resolved before the flow finishes.

Requirements

• My PR is focused on a single feature addition or bugfix.
• My PR has relevant, comprehensive unit tests.
• My PR is on a custom branch (i.e. is not named main).

Example code

import time

from quacc import flow, job


@job
def slow_math():
    # An independent piece of work that might take a while
    time.sleep(2)
    return 5


@flow()
def math_flow():
    return slow_math()


@flow()
def double_flow():
    a = math_flow()
    b = math_flow()
    return a + b


math_flow()
double_flow()

Before these patches, prefect would throw warnings about how tasks were not being resolved within flows. After, each flow finishes cleanly.

---

Summary of Changes

>> Provide context and a description of your changes here. Make sure to reference any associated issues. <<

Requirements

• My PR is focused on a single feature addition or bugfix.
• My PR has relevant, comprehensive unit tests.
• My PR is on a custom branch (i.e. is not named main).

Note: If you are an external contributor, you will see a comment from @buildbot-princeton. This is solely for the maintainers.