Repository navigation
🖥️ Desktop: dimos gateway - #4429
Draft
jeff-hykin wants to merge 262 commits into
Draft
jeff-hykin wants to merge 262 commits into
jeff-hykin wants to merge 262 commits into
Conversation
Contributor
There was a problem hiding this comment.
go2 replay realtime (arm64)
Details
| Benchmark suite | Current: 92d7ece | Previous: a16e03a | Ratio |
|---|---|---|---|
peak memory |
1834.023 MB |
1832.699 MB |
1.00 |
peak threads |
366 threads |
360 threads |
1.02 |
network (transport) |
2724.068 MB |
2731.039 MB |
1.00 |
disk write |
1.492 MB |
1.699 MB |
0.88 |
instructions |
136.158 G |
137.742 G |
0.99 |
This comment was automatically generated by workflow using github-action-benchmark.
…uns/restart, a launch's overrides, steps and problems, /dimos/paths server; fixture refreshed from Desktop
…own cloud code (no copied device flow, no message matching) cli/cloud.py's device flow and whoami become functions the CLI and the server's worker share; a missing or refused key raises NotLoggedInError, so the worker classifies by type and HTTP status. The console link and what counts as a recording (kind_of) come from dimos.cloud.
…ay dimos run does, and is tested on real blueprints get_by_name / get_blueprint_by_name / get_module_class_by_name instead of a copy of them, active_blueprints with no getattr fallbacks (the catalog used every atom, the rest only active ones), and a test that every ModuleConfig field is hidden or shown on purpose.
…nt_names, not a copy of its demo- rule)
…and their essential args, the app to install, starter, hidden), with a schema; the all_blueprints test keeps it in step with the code
…ir args inlined, tags from modes, registered/unlisted); dimos.yaml names the file; API 1.1.0
…s run registry and LOG_DIR, and logs are tested with what dimos's logger writes A run's log is found through its registry entry (then the log folders), is_pid_alive and REGISTRY_DIR are imported instead of copied, and registry_runs takes RegistryRun's fields from each RunEntry.
…, stream directions, dimos.yaml's version), and the checkout version read with tomllib
…ror) instead of skipping it silently every second
…flags and GlobalConfig, and passed as --key=value PUT /dimos/global-config, a launch and a restart refuse a key that isn't one of the root flags dimos builds from GlobalConfig (a typo, or a field renamed since it was saved) or a value GlobalConfig refuses, with a 400 naming it, instead of dimos failing at startup. A test runs every flag through dimos's real root callback and argv handling; --key=value keeps "--simulation ''" from being read as a bare --simulation (mujoco).
…nect alone (it already reads ZENOH_CONNECT; no second reader)
… processes, cached on disk by commit + packages), modules ranked per robot, module config with json_compatible, custom-robot docs, extras and install jobs
…ry.py does), so mypy resolves it
…s only writer) and written in a thread
…er starts python slowly)
…e; CI tests 3.10)
… robot listing it, else the one whose dirs hold it), not a hardcoded list of vendor folders
… loader, extras from metadata by packaging markers, a job's failure is its exit code and output tail (no wording), rescan whenever an importable package, Python file or packaging file changed, one venv helper, docs links tested against real pages; /dimos/paths recordingsDir defaults to dimos's RECORDINGS_DIR
…s structured log (stage/problem fields, exception_fields, typed errors at the source), not console text; models derived from dimos's RunEntry and StreamRef
…today), not only a list: a logged-in account answered 500
…es} checked against dimos's schemas, module flags after run <bp>, saved module config per blueprint via PUT /dimos/blueprints/{name}/config) and secrets (env not argv, shown as •••, kept for restart in an owner-only file)
…arted (startedAt) and the zenoh namespace it publishes its events under (null = SSE only), so Desktop restarts it when its code changes and stops relaying its events twice; recordingsDir falls back to Desktop's $DIMOS_RECORDINGS_DIR before dimos's own; API 1.3.0
…nd the secrets fields: fixture refreshed from Desktop's GET /api/desktop/openapi, no known differences left for them
…nitree-dds, dds on python > 3.10: no wheel) first finds the CycloneDDS C library it builds against: $CYCLONEDDS_HOME, else nix builds the one dimos's flake.lock pins (out-link in the venv, a GC root), else Homebrew's; none = the job fails before uv with code cyclonedds_missing; jobs get a code (API 1.4.0)
…ith a skill flag (/catalog/rpcs), /runs/rpcs lists the running ones and POST /runs {rpc} calls one directly (API 6.0.0)
…es (a Literal's or an Enum's values)
… per blueprint or module)
… answers them, instead of each route wrapping them in ApiError(status, str(error))
…eams through one Dimos.connect()) and /history replace /runs; the gateway saves no config (each launch carries its own, GET /catalog/config has the schema); dimos run --run-id is gone; the how-to moves to docs/usage/gateway.md
…ree-go2-pgo and unitree-go2-relocalization
…d) and start (its log's first record), since dimos stop drops its registry entry
…e name (no wrapper)
…tart/unload with the instance's class (TODO(core) where an instance name is needed)
…them; a test keeps its paths equal to the served OpenAPI)
…robots: section, read by annotations.py (was robots.py)
This branch was successfully deployed
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
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.
The stable API for building apps on top of dimos, served by Desktop at /dimos-gateway/ (API 7.0.0, no cloud): /catalog, /active, /history, extras, jobs.