-
Notifications
You must be signed in to change notification settings - Fork 55
Description
Is your feature request related to a problem? Please describe.
#179 added the support of input_shields on initializing AsyncAgent. We would like to have the support of output_shields as well.
Describe the solution you'd like
The current code added with #179 assumes all sheilds defined are input shields.
We need to have a way to distinguish input and output shields as shields defined in a run file do not seem to have a clear indicator to identify that it should be used for input or output.
Here are the naming convention of shield_id that I propose for that purpose:
- If the
shield_idstarts withinput_, it will be used for input only. - If the
shield_idstarts withoutput_, it will be used for output only. - If the
shield_idstarts withinout_, it will be used both for input and output. - Otherwise, it will be used for input only.
Using this convention, build list for input_shields and output_shields, then they are passed to
the get_agent method and and AsyncAgent is instantiated accordingly.
Describe alternatives you've considered
While instances of agent are internally created as a part of lightspeed-stack API call, we may want to have a separate API just to instantiate an agent.
Additional context
n/a