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

typo bugfixes in getting started with prompts #651

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

charlesfrye
Copy link
Contributor

tl;dr: input -> word, output -> antonym, rename to dynamic_prompt consistently

The provided code in this example doesn't run, because the keys are word and antonym, rather than input and output.

Also, the ExampleSelector-based prompt is named few_shot_prompt when defined and dynamic_prompt in the follow-up example. The former name is less descriptive and collides with an earlier example, so I opted for the latter.

Thanks for making a really cool library!

The provided code doesn't run, because the keys are `word` and `antonym`, rather than `input` and `output`.

Also, the `ExampleSelector`-based prompt is named `few_shot_prompt` when defined and `dynamic_prompt` in the follow-up example. The former name is less descriptive and collides with an earlier example, so I opted for the latter.

Thanks for making a really cool library!
Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

thank you!

@hwchase17 hwchase17 merged commit bfb23f4 into langchain-ai:master Jan 19, 2023
Blaizzy added a commit to Blaizzy/langchain that referenced this pull request Apr 18, 2024
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