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

Prevent IndexError in collect_playbook_responses when not in playbook #244

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

SeanScripts
Copy link
Collaborator

When the playbook invokes a flow and ends the turn/session in that flow, collect_playbook_responses was getting an IndexError. This should fix the error in #243

I think we still need to add flow invocations in this evaluation script, though.

@SeanScripts SeanScripts requested a review from kmaphoenix October 2, 2024 21:40
Copy link
Member

@kmaphoenix kmaphoenix left a comment

Choose a reason for hiding this comment

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

LGTM

@kmaphoenix kmaphoenix merged commit 6476985 into main Oct 7, 2024
7 checks passed
@kmaphoenix kmaphoenix deleted the feature/playbook_testing_update branch October 7, 2024 06:26
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.

2 participants