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 SQUALL few-shot testing script (Codex) #9

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

haileyschoelkopf
Copy link
Collaborator

modifying Spider few-shot script to have a version working with SQUALL.

Current issue with preprocessing script for SQUALL:

  • in process_squall_example function, original_col_name_dict is not a defined variable. I'm assuming the reason it doesn't work is because this fn was refactored into process_squall_example and build_column_name_dict (where original_col_name_dict variable is defined) but not sure what to change this to in process_squall_example, still looking.

@haileyschoelkopf
Copy link
Collaborator Author

Modified squall preprocessing. Now getting many "no such column" errors, followed by an eventual

sqlite3.ProgrammingError: Incorrect number of bindings supplied. The current statement uses 2, and there are 0 supplied

error message.

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