Skip to content

Ivan/chore/bump babel docs#3172

Open
leshy wants to merge 3 commits into
mainfrom
ivan/chore/bump-babel-docs
Open

Ivan/chore/bump babel docs#3172
leshy wants to merge 3 commits into
mainfrom
ivan/chore/bump-babel-docs

Conversation

@leshy

@leshy leshy commented Jul 24, 2026

Copy link
Copy Markdown
Member

fixes local links for docs, bumps md-babel-py

@mintlify

mintlify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
dimensional 🟢 Ready View Preview Jul 24, 2026, 9:49 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@greptile-apps

greptile-apps Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Updates the memory documentation and its executable examples for md-babel-py 1.3.2.

  • Bumps and locks md-babel-py 1.3.2, allowing it through uv's package-age exclusion.
  • Adds output suppression metadata and refreshes captured stream results.
  • Replaces externally hosted memory diagrams with relative asset references and reorders generated plot outputs.

Confidence Score: 3/5

The PR should not merge until the documentation images remain available to Mintlify; the plotting example should also be realigned with its stated threshold.

Relative references now point the published pages at Git LFS assets that Mintlify does not fetch, while one executable plot uses a 1.5-second threshold despite documenting and displaying the 7.5-second behavior.

Files Needing Attention: docs/capabilities/memory/index.md, docs/capabilities/memory/plot.md, docs/capabilities/memory/algo_comparison.md

Important Files Changed

Filename Overview
docs/capabilities/memory/index.md Refreshes example output and moves images to local LFS-backed paths that the documented Mintlify deployment cannot fetch.
docs/capabilities/memory/plot.md Updates Babel metadata and image references but introduces a plotting threshold inconsistent with the prose and displayed output.
docs/capabilities/memory/algo_comparison.md Suppresses code-block output and moves generated plots from external URLs to affected local LFS paths.
pyproject.toml Raises the md-babel-py requirement to 1.3.2 and deliberately exempts it from uv's seven-day publication delay.
uv.lock Consistently locks md-babel-py 1.3.2 and synchronizes its requirements across dependency groups.

Reviews (1): Last reviewed commit: "bump md-babel" | Re-trigger Greptile

our drawing system applies turbo color scheme to timestamps by default

![output](https://raw.githubusercontent.com/dimensionalOS/dimos-docs-assets/main/capabilities/memory/assets/color_image.svg)
![output](assets/color_image.svg)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Mintlify cannot load LFS images

When these pages are published through Mintlify, the relative URLs resolve to docs/capabilities/memory/assets, which is tracked through Git LFS even though the deployment does not fetch LFS content, causing the memory documentation images to render as unavailable or invalid pointer content.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

color=color.green,
gap_fill=0.0,
connect=7.5
connect=1.5

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Plot threshold contradicts tutorial

This example now uses a 1.5-second connection threshold while the prose, later related example, and displayed generated output use 7.5 seconds, so readers running the snippet receive a more fragmented plot than the documentation describes.

@codecov

codecov Bot commented Jul 24, 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    #3172      +/-   ##
==========================================
+ Coverage   74.02%   74.16%   +0.14%     
==========================================
  Files        1105     1108       +3     
  Lines      103722   104052     +330     
  Branches     9500     9530      +30     
==========================================
+ Hits        76777    77174     +397     
+ Misses      24225    24132      -93     
- Partials     2720     2746      +26     
Flag Coverage Δ
OS-ubuntu-24.04-arm 67.65% <ø> (+0.17%) ⬆️
OS-ubuntu-latest 69.72% <ø> (+0.16%) ⬆️
Py-3.10 69.71% <ø> (+0.15%) ⬆️
Py-3.11 69.71% <ø> (+0.15%) ⬆️
Py-3.12 69.72% <ø> (+0.17%) ⬆️
Py-3.13 69.72% <ø> (+0.16%) ⬆️
Py-3.14 69.72% <ø> (+0.16%) ⬆️
Py-3.14t 69.72% <ø> (+0.17%) ⬆️
SelfHosted-Large 29.65% <ø> (-0.03%) ⬇️
SelfHosted-Linux 36.22% <ø> (-0.05%) ⬇️
SelfHosted-macOS 35.23% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 6 files 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.

@mintlify

mintlify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
dimensional 🟡 Building Jul 24, 2026, 9:48 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

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.

1 participant