Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bedrock Agent required arguments documented as optional #89

Open
rvernica opened this issue Nov 3, 2024 · 0 comments
Open

Bedrock Agent required arguments documented as optional #89

rvernica opened this issue Nov 3, 2024 · 0 comments

Comments

@rvernica
Copy link

rvernica commented Nov 3, 2024

The following two Bedrock Agent parameters are required in the configuration file but there are documented as optional.

Required arguments in the agenteval.yml file when target type: bedrock-agent:

  bedrock_session_attributes: {}
  bedrock_prompt_session_attributes: {}

Both are listed as optional in the documentation file.

If not specified in the agenteval.yml the following error is reported:

...
  File "/agent-evaluation/src/agenteval/plan/plan.py", line 194, in _run_test
    target = self._target_factory.create()
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/agent-evaluation/src/agenteval/targets/target_factory.py", line 42, in create
    return target_cls(**{k: v for k, v in self.config.items() if k != "type"})
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: BedrockAgentTarget.__init__() missing 2 required positional arguments: 'bedrock_session_attributes' and 'bedrock_prompt_session_attributes'

The BedrockAgentTarget constructor has the two arguments with no defaults.

This issue is reported against the main branch at the linked revision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant