Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

README.md Hyperlink Fixes #531

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
ashjorda wants to merge 2 commits into awslabs:main
base: main
Choose a base branch
Loading
from ashjorda:ashjorda-patch-1
Open

Conversation

@ashjorda
Copy link

@ashjorda ashjorda commented Oct 21, 2025
edited
Loading

Amazon Bedrock AgentCore Samples Pull Request

Important

  1. We strictly follow a issue-first approach, please first open an issue relating to this Pull Request.
  2. Once this Pull Request is ready for review please attach review ready label to it. Only PRs with review ready will be reviewed.

Issue number: #530

Concise description of the PR

Removed old MemoryClient travel advisor agent and replaced with MemoryManager and AgentCoreMemorySessionManager to showcase a more streamlined approach
Updated Math and personal fitness short term memory examples to utilize MemoryManager

User experience

Prior to this code change developers were getting examples for the previous memory management experience. Developers will not have a sufficient amount of examples to utilize MemoryManager which seperates data plane from control plane for memory management.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • [ X] I have reviewed the contributing guidelines
  • [ X] Add your name to CONTRIBUTORS.md
  • [ X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Are you uploading a dataset?
  • Have you documented Introduction, Architecture Diagram, Prerequisites, Usage, Sample Prompts, and Clean Up steps in your example README?
  • [ X] I agree to resolve any issues created for this example in the future.
  • [ X] I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Acknowledgment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /