Skip to content

Conversation

@eywalker
Copy link
Contributor

Enhance various stream and source methods to accept execution_engine_opts, improving the flexibility and configurability of execution engines within the Orcapod framework. This update includes modifications to core classes and protocols to support the new options.

- Enhanced InvocationBase and SourceBase classes to accept `execution_engine_opts` in methods like `iter_packets`, `as_table`, `flow`, `run`, and `run_async`.
- Updated StatefulStreamBase and its subclasses to support `execution_engine_opts` in their method signatures and implementations.
- Modified CachedPodStream, KernelStream, LazyPodResultStream, PodNodeStream, TableStream, and WrappedStream to propagate `execution_engine_opts` through their respective methods.
- Updated RayEngine to handle function submission with engine-specific options.
- Enhanced the Pipeline class to pass `execution_engine_opts` during node execution.
- Updated core protocols to include `execution_engine_opts` in relevant method signatures for Pods and Streams.

This change improves flexibility and configurability of execution engines across the Orcapod framework.
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@brian-arnold brian-arnold self-requested a review November 2, 2025 01:52
Copy link
Collaborator

@brian-arnold brian-arnold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Went through everything and I see where resource options ultimately are given to the ray decorated function. It's a much bigger update than I would have expected!

@eywalker eywalker merged commit 7d39817 into main Nov 4, 2025
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants