Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workflow Fixes #156

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Workflow Fixes #156

merged 1 commit into from
Apr 5, 2024

Conversation

ykeremy
Copy link
Contributor

@ykeremy ykeremy commented Apr 5, 2024

- Update `get_all_parameters` to include any parametrizable inputs
  • for example TaskBlock.url can be reference to another parameter, if it is TaskBlock.get_all_parameters should return that parameter as well
  • In order to fix workflow status handling, now Block.execute returns a BlockResult
  • Implement Block.execute_safe
  • Allow LLM to return other actions alongside COMPLETE
  • Don't discard other actions if LLM returns other actions alongside COMPLETE
  • Implement DownloadFileAction
  • Implement UploadToS3
  • Allow passing a directory to send email block

- Update `get_all_parameters` to include any parametrizable inputs
  - for example TaskBlock.url can be reference to another parameter, if it is TaskBlock.get_all_parameters should return that parameter as well
- In order to fix workflow status handling, now Block.execute returns a BlockResult
- Implement Block.execute_safe
- Allow LLM to return other actions alongside COMPLETE
- Don't discard other actions if LLM returns other actions alongside COMPLETE
- Implement DownloadFileAction
- Implement UploadToS3
- Allow passing a directory to send email block
@ykeremy ykeremy added the sync label Apr 5, 2024
@ykeremy ykeremy merged commit 0800990 into main Apr 5, 2024
2 checks passed
@ykeremy ykeremy deleted the ykeremy/workflow-fixes branch April 5, 2024 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant