testing markdown eval in CI #939
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| ((needs.dev.result == 'success') || | ||
| (needs.dev.result == 'skipped')) | ||
| }} | ||
| cmd: "pip install md-babel-py && md-babel-py run README.md && md-babel-py run docs/*.md && git diff --exit-code" |
Contributor
There was a problem hiding this comment.
Good idea.
But it doesn't work currently. It can't execute the bash code in README.md
- Image.py: Import Any - Transform.py: Add type: ignore for parent_frame/child_frame kwargs - PointCloud2.py: Add type: ignore for to_rerun() and fill_mode - OccupancyGrid.py: Add type: ignore for to_rerun() - TFMessage.py: Add type: ignore for untyped to_rerun() call Former-commit-id: 3176890 [formerly 0407ed9] Former-commit-id: 57307e0
- AbstractImage: Add @AbstractMethod to_rerun() -> Any - OccupancyGrid: Add Any to typing imports At this point I dont even know anymore. Former-commit-id: 6467c96 [formerly 918ac21] Former-commit-id: e0c33cd
* fix: Make path visualization thicker and elevated above floor - Add radii parameter (default 0.05m = 5cm thick) - Add z_offset parameter (default 0.2m above floor) - Prevents path from being occluded by costmap mesh Path now clearly visible in Rerun 3D viewer. * feat:some rerun visualization improvements - Fix costmap colors: use turbo colormap, remove broken image panel, increase z_offset to 0.05 (turbo is bullshit i will fix this) - Fix websocket_vis: use global_config instead of os.environ for viewer_backend - Add TFRerunModule: auto-visualize all TF transforms via LCM subscription - Add autolog_to_rerun_async(): async logging with background thread for Out streams - Add log_timing_to_rerun(): decorator for timing metrics in Rerun - Wire TFRerunModule into go2 blueprints Addresses feedback from issue #922 * fix: Use Foxglove-style colors for costmap (blue-purple free, black occupied) - Free space: #484981 (blue-purple) - Occupied: gradient to #000000 (black) - Replaces incorrect turbo colormap * fix: Remove autolog_to_rerun from A* planner, use manual logging - Removes dependency on stream.py autolog_to_rerun() method - Uses manual Rerun logging like CostMapper/VoxelGridMapper - Keeps stream.py clean without Rerun-specific code * fix: Subscribe to internal planner path, not module Out stream Out streams don't have subscribe() method - fixed to use self._planner.path * style: Apply pre-commit formatting and linting fixes * fix: correct stream.py * refactor: Remove all Rerun code from stream.py - Remove import rerun - Remove autolog_to_rerun() and _log_to_rerun() methods - Remove _rerun_config and _rerun_last_log from __init__ - Keep Out.subscribe() and ObservableMixin (core functionality) * style: Apply linter formatting * fix: Address mypy errors from Rerun changes - Add type ignore for to_rerun() calls - Remove Disposable() double-wrapping, add subscriptions directly - Move type ignore comment for Path.to_rerun() * fix: Correct camera image format from BGR to RGB Frame is decoded as rgb24 but Image.from_numpy() was defaulting to BGR, causing red/blue color swap in Rerun camera feed * fix: Camera RGB color swap and additional cleanup - Fix camera image format: specify RGB when decoding rgb24 frames - Wrap module input subscriptions with Disposable() - Add type ignores for untyped to_rerun() calls * CI code cleanup Former-commit-id: 48583fc [formerly fca9a14] Former-commit-id: 55e6145
* modules docs sketch * extracted introspection tooling * first sketch of module docs * including go2 basic blueprint svg * new dot2 generator * adding different layouts * removing other blueprint rendering algos * better dot2 algo * modules tutorial work * more docs work * small modules docs changes * go2 agentic svg * mutliprocessing sketch * small wording changes * config docs, moved old docs, dot file included * doclinks tool * config docs moved to generic place * doclinks updates, transform docs sketch * transforms docs * docs agent docs, reactivex docs * reactivex docs * small changes on transform images * folded code blocks * moved modules doc * transforms work * transform docs * transforms modules image * Update docs/concepts/modules.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * Update docs/concepts/modules.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * Update docs/concepts/modules.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * Update docs/concepts/modules.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * Update docs/concepts/modules.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * doclinks ignore * doclinks in pre-commit hook * camera module fixes * vis cleanup * doclinks always runs * introspection tooling cleanup * kwargs typing * paul comment * lcm and transports docs * camera module fixes * small cleanup, vibed sensor.py deleted * sharpness barrier fix * moved gstreamer into separate dir, removed fake zed * removed all typing ignores * mypy fix * Out stream is observable * better video stream skill * stream changes undo, typing fixes * reverted pyproject for fast tests in CI for now * CI code cleanup * type fixes * ignore gps skill publish * transform test fix * core test fix * watchdog was missing --------- Co-authored-by: Paul Nechifor <paul@nechifor.net> Former-commit-id: 929d719 [formerly 4499e08] Former-commit-id: dd489f6
* reactivex docs * sensor stream docs, alignment, quality filter * pulled some stuff into advanced_streams.md * closing folded tags * folding codeblocks * sensor storage and data storage docs * comment on module replay * quality filtering docs improvement * sharpness graph * quality filter mosaic * oops * pushed lfs frame mosaic * small improvements * Update docs/api/sensor_streams/index.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * Update docs/api/sensor_streams/reactivex.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * nicer quality filter * fold vis * small wording fix * Update docs/api/sensor_streams/reactivex.md Co-authored-by: Paul Nechifor <paul@nechifor.net> * removed fancy ref counting docs * Update docs/api/sensor_streams/temporal_alignment.md Co-authored-by: Paul Nechifor <paul@nechifor.net> --------- Co-authored-by: Paul Nechifor <paul@nechifor.net> Former-commit-id: 662ddbc [formerly 6c1c971] Former-commit-id: 33ff4bb
Remove paths-ignore for **.md so the workflow triggers on markdown changes. The internal dorny/paths-filter still prevents unnecessary docker rebuilds - only check-markdown runs on markdown-only changes. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Former-commit-id: 889cb3f [formerly 6ea00e7] Former-commit-id: ea735d9
Use the tests.yml workflow to run md-babel-py inside the dimos dev container instead of the standalone md-babel-py Docker image. This ensures all project dependencies (curl, git, python packages, etc.) are available when executing code blocks. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Former-commit-id: 6962d26 [formerly 386f736] Former-commit-id: 0d44bc6
|
Too many files changed for review. |
|
Too many files changed for review. |
|
Too many files changed for review. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
testing auto eval of all markdown code blocks, fail if output changes