Closed marcusschiesser closed 4 months ago
Latest commit: a5d137e245a9f4fa8e646e9016014533008b54ef
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
The recent updates focus on enhancing configuration flexibility in a TypeScript chat engine project. New environment variables for controlling the language model's behavior have been introduced and organized into a dedicated function. Additionally, the assignment logic for a key chat engine parameter now dynamically adapts based on environment settings, improving customization without altering the default behavior.
Files | Change Summary |
---|---|
helpers/env-variables.ts |
Introduced LLM_TEMPERATURE and LLM_MAX_TOKENS . Moved these variables to getEngineEnvs for better clarity. |
.../typescript/chat/chat.ts |
Modified similarityTopK assignment to use TOP_K environment variable or default to 3. |
🐇💻 A whimsical ode by CodeRabbit 🎉
In the land of code, where the bits align, New env vars dance, definitions refine.
TOP_K
takes a bow, with a dynamic spin, Defaulting to three, where no env begins. Cheers to the devs, with each clever commit, Crafting the future, bit by byte, bit by bit! 🌟🚀
helpers/env-variables.ts (2)
`166-173`: LGTM! The addition of `LLM_TEMPERATURE` and `LLM_MAX_TOKENS` is correctly implemented. --- `213-222`: LGTM! The `getEngineEnvs` function correctly sets up the `TOP_K` environment variable.
Summary by CodeRabbit
similarityTopK
property to dynamically use environment settings, ensuring more flexible configuration.