Remove duplicate `params` & `context` args

dexaai / dexter

LLM tools used in production at Dexa

https://dexter.dexa.ai

MIT License

69 stars 4 forks source link

Remove duplicate `params` & `context` args #14

Closed rileytomasek closed 9 months ago

rileytomasek commented 9 months ago

These are both already handled statefully by the ChatModel so we shouldn't be introducing a second way to accomplish the same thing and more arguments to support it.

Remove duplicate context arg
Remove duplicate params argument

vercel[bot] commented 9 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
dexter	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 3, 2023 0:19am

rileytomasek commented 9 months ago

Good catch. I will change it to use clone until you get around to transitioning from being mutable to using extend immutably.