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

Add helper to run a query and select the response #56

Merged
merged 6 commits into from
Apr 20, 2024

Conversation

jaresty
Copy link
Collaborator

@jaresty jaresty commented Apr 19, 2024

No description provided.

GPT/gpt.py Outdated Show resolved Hide resolved
GPT/gpt.talon Outdated Show resolved Hide resolved
@C-Loftus
Copy link
Owner

This looks like a good idea. I like how it mimics statefulness without needing to explicitly add state internally. I made a few comments then I think we can merge?

@C-Loftus
Copy link
Owner

Thanks for making edits. I will test this out locally later/tomorrow, perhaps add some docs, change the old clip command, then merge

@C-Loftus
Copy link
Owner

Tested. Merging this! Thanks for this idea, I think it will be useful! I am curious to continue thinking about other results / pipelines for model outputs (i.e. we have select, clip, and I wonder if there are other ways to process that would be useful. Will continue to brainstorm for future PRs)

@C-Loftus C-Loftus merged commit 04d9435 into main Apr 20, 2024
2 checks passed
@C-Loftus C-Loftus deleted the add-paste-selected-helper branch April 20, 2024 02:56
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