You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
great idea. prs are def being accepted! will keep an eye out for it. at the same time, this is such a good idea im more than happy to work on myself if you dont have the time - lmk
Addresses the issue in #76 by either using the relevant environment
variable if set or using a string passed in the constructor.
Prefers the constructor string over the environment variable, which
seemed like the natural choice to me.
It's just a bit annoying, I want to use this library in production and I currently store credentials not in the environment.
I think the ideal API is like all the AWS SDKs where you can either stick them in the environment OR pass them as params to the llm constructor.
I can do a PR for this if you're accepting PRs?
The text was updated successfully, but these errors were encountered: