Skip to content

Adding the minor feedback give by Dustin Liu for MCP server lab #81

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rajjainl
Copy link

Issue #, if available:

Description of changes:

  1. Changed the image to remove the AWS Documentation MCP Server to AWS CDK Server in the 01-tutorials/01-fundamentals/04-tools/01-using-mcp-tools/mcp-agent.ipynb

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

rajjainl added 2 commits June 20, 2025 19:40
Added image for Dustin feedback
Adding Feedback by Dustin to add the image changes in code repository for the image
@rajjainl rajjainl changed the title Adding the minor feedback give by Dustin Liu Adding the minor feedback give by Dustin Liu for MCP server lab Jun 21, 2025
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see read lines in the architecture on text please remove

@@ -350,7 +350,7 @@
"With Strands Agents you can also interact with multiple MCP servers using the same agent and configure tools setups such as the max number of tools that can be used in parallel (`max_parallel_tools`). Let's create a new agent to show case this configuration:\n",
"\n",
"<div style=\"text-align:center\">\n",
" <img src=\"images/architecture_3.png\" width=\"85%\" />\n",
" <img src=\"images/Architecture_3_CDK_MCP_Server.png\" width=\"85%\" />\n",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please keep the name architecture_3.png

@EashanKaushik EashanKaushik self-assigned this Jun 23, 2025
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

Successfully merging this pull request may close these issues.

2 participants