Description

This is related to #155 which reported errors when dill attempted to serialize the closure task wrappers for failure task injection. These closures were serialized by value which caused issues with type annotations containing pydantic models which use ABCs, and dill 0.3.6, required by Globus Compute, can't pickle ABCs.

This PR addresses two aspects of this problem:

• Task data structures were reverted from pydantic BaseModels to dataclasses. This reverts Create Config docstrings from Field descriptions #137.
• Closures are no longer used to wrap tasks in the failure app.

I also caught a bug in how _dependency_failure_parent_task was called. It was used like a task factory, but was incorrectly wrapped in the @task decorator.

Fixes

• Fixes Get parsl.serialize.errors.SerializationError when using Parsl htex to run docking and moldesign #155

Type of Change

• Bug (non-breaking change which fixes an issue)
• Enhancement (non-breaking change which adds or improves functionality)
• Internal (refactoring, performance, and testing)
• Breaking (fix or feature that would cause existing functionality to not work as expected)
• Documentation (no changes to the code)
• Development (CI workflows, packages, templates, etc.)
• Package (package dependencies and versions)

Testing

Tested the cholesky app alone and with failure injection using Dask, Process Pool, Ray, and Parsl. I also tested the failure modes for dependency and import which have different task wrappers.

Pull Request Checklist

Please confirm the PR meets the following requirements.

• Relevant tags are added (breaking, bug, documentation, enhancement, package, etc.).
• Code changes pass pre-commit (e.g., ruff, mypy, etc.).
• Tests have been added to show the fix is effective or that the new feature works.
• New and existing unit tests pass locally with the changes.
• Docs have been updated and reviewed if relevant.