Skip to content

chore(examples/python): upgrade moss package to 1.1.1 to resolve on-d…#264

Merged
HarshaNalluru merged 1 commit into
usemoss:mainfrom
riit3sh:chore/upgrade-moss-requirement
May 28, 2026
Merged

chore(examples/python): upgrade moss package to 1.1.1 to resolve on-d…#264
HarshaNalluru merged 1 commit into
usemoss:mainfrom
riit3sh:chore/upgrade-moss-requirement

Conversation

@riit3sh
Copy link
Copy Markdown
Contributor

@riit3sh riit3sh commented May 27, 2026

Pull Request Checklist

Please ensure that your PR meets the following requirements:

  • I have read the CONTRIBUTING guide.
  • I have updated the documentation (if applicable).
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.

Description

This PR bumps the moss package dependency in the examples/python directory from 1.1.0 to 1.1.1.

Context & Motivation:

  • In moss 1.1.0, running load_and_query_sample.py raised a connection timeout during the local prewarm stage: Query error: failed to load embedding model 'moss-minilm': model download failed: error decoding response body: operation timed out.
  • Additionally, falling back to a cloud search resulted in 503 Service Unavailable errors.
  • Upgrading to the newly released moss 1.1.1 completely resolves these model downloading and fallback issues. Bumping this dependency ensures that any new developer testing the Python examples gets a working out-of-the-box experience with local queries returning successfully in under 2ms as expected.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Open in Devin Review

Copilot AI review requested due to automatic review settings May 27, 2026 18:57
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 27, 2026

CLA assistant check
All committers have signed the CLA.

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

Note

Copilot was unable to run its full agentic suite in this review.

Bumps the minimum required version of the moss dependency in the Python examples.

Changes:

  • Update moss minimum version from 1.1.0 to 1.1.1 in examples/python/requirements.txt

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

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@HarshaNalluru
Copy link
Copy Markdown
Contributor

@riit3sh thanks for the contribution, could you please sign the CLA at #264 (comment)?

@riit3sh
Copy link
Copy Markdown
Contributor Author

riit3sh commented May 27, 2026

@HarshaNalluru Done! Just signed the CLA and verified that the check has passed. Thanks for the ping!

@HarshaNalluru HarshaNalluru merged commit d2f6cf5 into usemoss:main May 28, 2026
13 checks passed
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.

4 participants