Uh oh!
There was an error while loading. Please reload this page.
Improve log output and environment initialization summary - #527
Merged
Conversation
Greptile SummaryThe PR modernizes console logging and replaces legacy environment startup messages with a structured initialization summary.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| embodichain/lab/gym/envs/base_env.py | Replaces legacy initialization messages with structured runtime, timing, metadata, robot, and sensor summary generation. |
| embodichain/lab/gym/envs/embodied_env.py | Defers summary output until managers and recording state are initialized, then includes manager and functor details. |
| embodichain/utils/logger.py | Adds UTC timestamp formatting, aligned columns, default level colors, custom color overrides, and optional plain records. |
| tests/gym/envs/test_initialization_summary.py | Verifies summary contents, metadata compaction, manager coverage, and logging behavior. |
| tests/utils/test_logger.py | Verifies timestamp layout, level colors, overrides, prefix omission, and formatted exceptions. |
Sequence Diagram
sequenceDiagram
participant Caller
participant EmbodiedEnv
participant Managers
participant Summary
participant Logger
Caller->>EmbodiedEnv: Initialize environment
EmbodiedEnv->>Managers: Initialize managers and recording state
Managers-->>EmbodiedEnv: Initialization complete
EmbodiedEnv->>Summary: Build runtime and manager summary
Summary-->>EmbodiedEnv: Structured summary lines
EmbodiedEnv->>Logger: Emit initialization summary
Reviews (4): Last reviewed commit: "wip" | Re-trigger Greptile
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
Description
This PR improves EmbodiChain console output and environment startup diagnostics:
DEBUGcyan,INFOgreen,WARNINGyellow, andERRORred);color=Noneto disable coloring;EmbodiedEnvsummary until managers and recording buffers are initialized;Representative output:
Dependencies: none. Issue: N/A.
Type of change
Screenshots
Not applicable; representative terminal output is included above.
Validation
black --check --diff --color ./— 674 files unchangedpytest tests/utils/test_logger.py -v— 10 passedpytest tests/gym/envs -m "not requires_sim" -q— 287 passed, 1 skipped, 22 deselectedpytest tests -q— 1594 passed, 107 skipped, 34 deselectedChecklist
black .command to format the code base.