model_context should be a public/accessible attribute in Agents #6026
jspv
started this conversation in
Feature suggestions
Replies: 1 comment
-
There wasn't any specific reason to make it private -- just being conservative. You can create a PR to make this a public property. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
This is a small one. Agent initiation allows one to pass in a model_context, but there isn't a "public: way to get the existing model_context created by default.
Use cases:
I find myself doing this regularly by accessing ._model_contex.add_message() .clear() .get_mesages() etc. Given that the underlying structure is not only exposed, it can be passed in, any reason not to create a public model_context property?
Beta Was this translation helpful? Give feedback.
All reactions