Skip to content

fix: release 13 issues#2671

Merged
paul-nechifor merged 1 commit into
mainfrom
paul/fix/release-13
Jul 1, 2026
Merged

fix: release 13 issues#2671
paul-nechifor merged 1 commit into
mainfrom
paul/fix/release-13

Conversation

@paul-nechifor

Copy link
Copy Markdown
Contributor

Problem

Since #2210 , we depend on rerun-sdk==0.32.0a1. But that's a pre-release. That means, people cannot do uv add dimos in their repo. If they do, they'll get dimos==0.0.12.post2. In order for them to get dimos==0.0.13 they have to change pyproject.toml to add:

[tool.uv]
prerelease = "allow"

This is not the case for pip. Doing pip install dimos installs 0.0.13.

Also, messed up the file inclusions and dimos/web/dimos_interface/api/server.py was not included.

Solution

  • Use rerun-sdk==0.32.0, since it's not a pre-release.
  • Add dimos/web/dimos_interface/api/server.py back.

@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates release packaging and the rerun SDK pin. The main changes are:

  • Includes the dimos_interface/api Python and HTML files in source distributions.
  • Narrows setuptools exclusions so the API subtree can be packaged.
  • Replaces the rerun-sdk prerelease pin with 0.32.0 in project metadata and the lockfile.
  • Refreshes the locked rerun package metadata and wheel hashes.

Confidence Score: 5/5

This looks safe to merge after checking the rerun viewer compatibility.

  • The packaging rules appear to include the API server and its template.
  • The remaining concern is the mixed stable SDK and alpha viewer pins in the visualization path.
  • No blocking issue was found in the changed code.

pyproject.toml

Important Files Changed

Filename Overview
MANIFEST.in Adds the API Python and HTML files back into the source distribution after the broader frontend exclusion.
pyproject.toml Narrows package exclusions and updates rerun-sdk to stable 0.32.0, while dimos-viewer remains pinned to the alpha release.
uv.lock Refreshes the locked rerun SDK package metadata, wheels, and dependency entries for 0.32.0.

Reviews (1): Last reviewed commit: "fix: release 13" | Re-trigger Greptile

Comment thread pyproject.toml
Comment thread pyproject.toml
@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #2671      +/-   ##
==========================================
- Coverage   70.82%   70.82%   -0.01%     
==========================================
  Files         903      903              
  Lines       81524    81524              
  Branches     7354     7354              
==========================================
- Hits        57740    57739       -1     
  Misses      21857    21857              
- Partials     1927     1928       +1     
Flag Coverage Δ
OS-ubuntu-24.04-arm 63.37% <ø> (+<0.01%) ⬆️
OS-ubuntu-latest 66.03% <ø> (-0.01%) ⬇️
Py-3.10 ?
Py-3.11 66.02% <ø> (+<0.01%) ⬆️
Py-3.12 ?
Py-3.13 66.02% <ø> (+<0.01%) ⬆️
Py-3.14 66.04% <ø> (-0.01%) ⬇️
Py-3.14t 66.02% <ø> (-0.01%) ⬇️
SelfHosted-Large 30.01% <ø> (ø)
SelfHosted-Linux 37.29% <ø> (ø)
SelfHosted-macOS 36.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Jul 1, 2026
@paul-nechifor paul-nechifor changed the title fix: release 13 fix: release 13 issues Jul 1, 2026
@paul-nechifor paul-nechifor enabled auto-merge (squash) July 1, 2026 02:23
@paul-nechifor paul-nechifor merged commit 74d6b9e into main Jul 1, 2026
61 of 64 checks passed
@paul-nechifor paul-nechifor deleted the paul/fix/release-13 branch July 1, 2026 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants