Ivan/chore/bump babel docs#3172
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Greptile SummaryUpdates the memory documentation and its executable examples for md-babel-py 1.3.2.
Confidence Score: 3/5The 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
Reviews (1): Last reviewed commit: "bump md-babel" | Re-trigger Greptile |
| our drawing system applies turbo color scheme to timestamps by default | ||
|
|
||
|  | ||
|  |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
Codecov Report✅ All modified and coverable lines are covered by tests. @@ 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
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
fixes local links for docs, bumps md-babel-py