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

Fix whitespace issues #10

Merged
merged 2 commits into from
Sep 18, 2024
Merged

Fix whitespace issues #10

merged 2 commits into from
Sep 18, 2024

Conversation

theoforger
Copy link
Owner

Some cards on the board can have more than one word, for example scuba diver. This could cause issue since the code used split_whitespace() to extract words.

This PR fixes this issue by changing the delimiter to , instead, and changing the system prompt accordingly.

Also changed the example files to use a real board of Codenames instead of randomly generated words.

@theoforger theoforger self-assigned this Sep 18, 2024
@theoforger
Copy link
Owner Author

GitHub Actions passed.

@theoforger theoforger merged commit d1536e4 into main Sep 18, 2024
1 check passed
@theoforger theoforger deleted the fix/whitespace branch September 18, 2024 15:50
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.

1 participant