Amazon Bedrock Agent Samples - Financial Analyst and Portfolio system - Multi-Agent example [Content Improvement]

[schinchli]

What are your suggestions?

Regarding Financial Analyst and Portfolio system - Multi-Agent example:

• Recommend to include libraries in requirement file : jmespath, urllib3>=1.26.0, matplotlib>=3.7.0 (please validate I know we are not plotting any charts - recommend testing if you can replicate)
• Folder name typo error: Fianancial Assistant Agent
• Recommend to add file to be uploaded in the folder itself (during clone) or in comments where do I get it from.
  In def upload_files(local_folder_path, S3_BUCKET_NAME),

upload_files(LOCAL_FOLDER_PATH, BUCKET_NAME) to be replaced with upload_files(LOCAL_FOLDER_PATH, S3_BUCKET_NAME) // Recommend you to check this as well.


Else this error came:
Cell In[19], line 24
21 except Exception as e:
22 print(f"Error uploading the file {str(e)}")
24 upload_files(LOCAL_FOLDER_PATH, BUCKET_NAME)
NameError: name 'BUCKET_NAME' is not defined. `

I'm unable to proceed further. Appreciate you can help me with next step(s).

[aswathsr101]

Hello Thank you for noting this.

I have gone ahead and rerun the code, not getting a Python libraries error. Probably the environment that you are running against is having this issue but happy to check it out.

Regarding the data the data can be obtained from online and uploaded : #129
have created a PR for this