Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add semantic memory rest api example#1143

Draft
o-love wants to merge 5 commits into
mainfrom
rest-api-notebook
Draft

Add semantic memory rest api example#1143
o-love wants to merge 5 commits into
mainfrom
rest-api-notebook

Conversation

@o-love
Copy link
Copy Markdown
Contributor

@o-love o-love commented Feb 26, 2026

No description provided.

@o-love o-love marked this pull request as ready for review February 26, 2026 23:03
Copilot AI reviewed Feb 26, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds a Jupyter notebook example demonstrating semantic memory REST API functionality, specifically focusing on set_metadata scoping. The PR includes the necessary dependencies (jupyter and nbconvert), a new CI workflow job to validate the notebook, and a configuration file for the CI test environment.

Changes:

  • Adds a new Jupyter notebook (rest_api_set_metadata.ipynb) demonstrating semantic memory set_metadata scoping with the REST API
  • Adds jupyter and nbconvert as development dependencies to support notebook examples
  • Adds a CI workflow job to automatically test the notebook execution against a live MemMachine server
  • Adds a CI configuration file for running MemMachine with OpenAI and PostgreSQL/pgvector

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
examples/rest_api_set_metadata.ipynb New notebook demonstrating semantic memory set_metadata scoping via REST API
pyproject.toml Adds jupyter>=1.1.0 and nbconvert>=7.16.0 to dev dependencies
uv.lock Lock file updates for new jupyter and nbconvert dependencies and their transitive dependencies
.github/workflows/pytest-integration.yml Adds new CI job to execute the notebook against a live MemMachine server
.github/ci/memmachine-openai-pgvector.yml Configuration file for CI testing with OpenAI and pgvector backend

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/ci/memmachine-openai-pgvector.yml
Comment thread .github/workflows/pytest-integration.yml
Comment thread .github/workflows/pytest-integration.yml
@o-love o-love force-pushed the rest-api-notebook branch from 52bcf40 to 70447e6 Compare March 2, 2026 19:53
@SarahScargall
Copy link
Copy Markdown
Contributor

SarahScargall commented Mar 2, 2026

Hi Oscar - can you create a README.md file for this example, explaining key details like process to follow, steps, minimum requirements, etc. A good example that you can work from can be found in /examples/openai_agent/README.md.

This will then help me ensure that documentation for your Example shows up on our docs.memmachine.ai site.

@sscargal sscargal added this to the v0.3.4 milestone Apr 6, 2026
sscargal
sscargal approved these changes Apr 10, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@sscargal sscargal left a comment

Choose a reason for hiding this comment

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

LGTM

@edwinyyyu
Copy link
Copy Markdown
Contributor

edwinyyyu commented Apr 10, 2026

Please fix uv.lock

@o-love o-love marked this pull request as draft April 10, 2026 22:25
@sscargal sscargal modified the milestones: v0.3.4, v0.3.5, v0.3.6 Apr 13, 2026
@sscargal sscargal removed this from the v0.3.6 milestone Apr 21, 2026
@sscargal
Copy link
Copy Markdown
Contributor

sscargal commented Apr 24, 2026

@copilot resolve the merge conflicts in this pull request

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 24, 2026 •
edited
Loading

Hi Oscar - can you create a README.md file for this example, explaining key details like process to follow, steps, minimum requirements, etc. A good example that you can work from can be found in /ex...

Added examples/rest_api_set_metadata/README.md in commit 88fd564. I also moved the notebook into its own subdirectory (examples/rest_api_set_metadata/) to be consistent with the other examples. The README covers prerequisites, how to start MemMachine, installation, configuration, a step-by-step walkthrough of how the example works, and troubleshooting tips.

Please fix uv.lock

Fixed in commit 88fd564. The uv.lock had a malformed TOML section introduced by the merge conflict (the babel wheel array was incorrectly mixed with an authlib entry). Regenerated the lock file with uv lock.

@copilot resolve the merge conflicts in this pull request

The merge conflict (in uv.lock) was resolved as part of the merge commit cc4cae3 that was already on this branch, but the resulting lock file had malformed TOML. That's now fixed in commit 88fd564 by regenerating the lock file cleanly.

Copilot AI requested a review from sscargal April 24, 2026 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@edwinyyyu edwinyyyu edwinyyyu approved these changes

@jealous jealous Awaiting requested review from jealous

@malatewang malatewang Awaiting requested review from malatewang

@sscargal sscargal Awaiting requested review from sscargal

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@o-love @SarahScargall @edwinyyyu @sscargal