Skip to content

[MISC] Reduce warning messages. #1294

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 11 commits into from
Jun 20, 2025
Merged

[MISC] Reduce warning messages. #1294

merged 11 commits into from
Jun 20, 2025

Conversation

YilingQiao
Copy link
Collaborator

@YilingQiao YilingQiao commented Jun 17, 2025 •
edited
Loading

Description

  • Catch the stderr mujoco when parsing URDF to reduce Geom with duplicate name
  • Delete some forced ti.loop_config(serialize=True) to avoid
[W 06/17/25 15:46:49.240 3574973] [frontend_ir.cpp:begin_frontend_struct_for_on_snode@1678] ti.loop_config(serialize=True) does not have effect on the struct for. The execution order is not guaranteed.
  • Make the warning message about actuator informative.

Related Issue

Resolves Genesis-Embodied-AI/Genesis#

Motivation and Context

How Has This Been / Can This Be Tested?

Screenshots (if appropriate):

Checklist:

  • I read the CONTRIBUTING document.
  • I followed the Submitting Code Changes section of CONTRIBUTING document.
  • I tagged the title correctly (including BUG FIX/FEATURE/MISC/BREAKING)
  • I updated the documentation accordingly or no change is needed.
  • I tested my changes and added instructions on how to test it for reviewers.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@YilingQiao YilingQiao marked this pull request as ready for review June 18, 2025 00:32
@YilingQiao YilingQiao requested a review from duburcqa as a code owner June 18, 2025 00:32
duburcqa
duburcqa reviewed Jun 18, 2025
View reviewed changes
duburcqa
duburcqa reviewed Jun 18, 2025
View reviewed changes
duburcqa
duburcqa reviewed Jun 18, 2025
View reviewed changes
duburcqa
duburcqa reviewed Jun 18, 2025
View reviewed changes
@duburcqa duburcqa changed the title [MISC] reduce warning messages [MISC] Reduce warning messages. Jun 18, 2025
duburcqa
duburcqa reviewed Jun 19, 2025
View reviewed changes
@duburcqa duburcqa enabled auto-merge (squash) June 20, 2025 17:54
@duburcqa duburcqa merged commit 0864103 into Genesis-Embodied-AI:main Jun 20, 2025
13 checks passed
duburcqa pushed a commit that referenced this pull request Jun 22, 2025
@YilingQiao @duburcqa
[MISC] Reduce warning messages. (#1294)
ce224a6
kanade1214 pushed a commit to kanade1214/Genesis that referenced this pull request Jul 8, 2025
@YilingQiao @duburcqa
[MISC] Reduce warning messages. (Genesis-Embodied-AI#1294)
0e0c79e
chris-la-humalab pushed a commit to HumaLab/Genesis that referenced this pull request Aug 1, 2025
@YilingQiao @chris-la-humalab
[MISC] Reduce warning messages. (Genesis-Embodied-AI#1294)
406d61b
YilingQiao added a commit to YilingQiao/Genesis that referenced this pull request Aug 22, 2025
@YilingQiao
[MISC] Reduce warning messages. (Genesis-Embodied-AI#1294)
0de6916
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@duburcqa duburcqa duburcqa approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@YilingQiao @duburcqa