Fixes parallelism action() typos

[skrawcz]

The signature was not correct.

Changes

How I tested this

Notes

Checklist

• PR has an informative and human-readable title (this will be pulled into the release notes)
• Changes are limited to a single goal (no scope creep)
• Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future TODOs are captured in comments
• Project documentation has been updated if adding/changing functionality.

---

Important

Fix incorrect method signature for action() in parallelism.rst by adding missing parameters state and inputs.

• Documentation:
  • Fix incorrect method signature for action() in TestMultiplePromptsWithSubgraph, TestMultiplePrompts, and another class in parallelism.rst.
  • Add missing parameters state: State and inputs: Dict[str, Any] to action() method signature.

^{This description was created by for 89c2862. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 89c2862 in 23 seconds

More details

• Looked at 31 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. docs/concepts/parallelism.rst:372

• Draft comment:
  Consider using Union from typing for compatibility with Python versions before 3.10.

def action(self, state: State, inputs: Dict[str, Any]) -> Union[Action, Callable, RunnableGraph]:

• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR correctly updates the method signature to include parameters that are likely required for the action method to function correctly. However, the return type hint should be updated to use the 'Union' from 'typing' for compatibility with Python versions before 3.10.

2. docs/concepts/parallelism.rst:478

• Draft comment:
  Consider using Union from typing for compatibility with Python versions before 3.10.

def action(self, state: State, inputs: Dict[str, Any]) -> Union[Action, Callable, RunnableGraph]:

• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR correctly updates the method signature to include parameters that are likely required for the action method to function correctly. However, the return type hint should be updated to use the 'Union' from 'typing' for compatibility with Python versions before 3.10.

3. docs/concepts/parallelism.rst:533

• Draft comment:
  Consider using Union from typing for compatibility with Python versions before 3.10.

def action(self, state: State, inputs: Dict[str, Any]) -> Union[Action, Callable, RunnableGraph]:

• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR correctly updates the method signature to include parameters that are likely required for the action method to function correctly. However, the return type hint should be updated to use the 'Union' from 'typing' for compatibility with Python versions before 3.10.

Workflow ID: wflow_Jn8plNTG20LsOJEI

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[github-actions]

A preview of is uploaded and can be seen here:

✨ https://burr.dagworks.io/pull/459 ✨

Changes may take a few minutes to propagate. Since this is a preview of production, content with draft: true will not be rendered. The source is here: https://github.com/DAGWorks-Inc/burr/tree/gh-pages/pull/459/