Skip to content

[release/3.0.0-beta2] Fix quadruped tutorial noise import#6037

Merged
ooctipus merged 1 commit into
isaac-sim:release/3.0.0-beta2from
ooctipus:fix/beta2-quadruped-noise-import
Jun 9, 2026
Merged

[release/3.0.0-beta2] Fix quadruped tutorial noise import#6037
ooctipus merged 1 commit into
isaac-sim:release/3.0.0-beta2from
ooctipus:fix/beta2-quadruped-noise-import

Conversation

@ooctipus

@ooctipus ooctipus commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

Source

Tests

  • git diff --check upstream/release/3.0.0-beta2..HEAD
  • Parsed scripts/tutorials/03_envs/create_quadruped_base_env.py with ast.parse

@github-actions github-actions Bot added bug Something isn't working isaac-lab Related to Isaac Lab team labels Jun 8, 2026
@greptile-apps

greptile-apps Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This backport fixes a broken import in the quadruped tutorial that would cause an ImportError at startup. The single-line change replaces AdditiveUniformNoiseCfg (which does not exist in isaaclab.utils.noise) with the correct UniformNoiseCfg.

  • Import fix: AdditiveUniformNoiseCfgUniformNoiseCfg (aliased as Unoise); the noise configurations used throughout ObservationsCfg are otherwise unchanged and remain valid.

Confidence Score: 5/5

Safe to merge — corrects a hard import error that prevented the tutorial from running at all.

The change is a single-line import correction. UniformNoiseCfg is confirmed to exist in isaaclab.utils.noise; AdditiveUniformNoiseCfg does not. All downstream usages of the Unoise alias are unchanged and valid. No logic, behavior, or API surface is modified.

No files require special attention.

Important Files Changed

Filename Overview
scripts/tutorials/03_envs/create_quadruped_base_env.py Corrects the noise class import from the non-existent AdditiveUniformNoiseCfg to the actual UniformNoiseCfg; the alias Unoise and all downstream usages are unchanged.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["create_quadruped_base_env.py"] --> B["from isaaclab.utils.noise import UniformNoiseCfg as Unoise"]
    B --> C["ObservationsCfg.PolicyCfg"]
    C --> D["base_lin_vel noise=Unoise(n_min=-0.1, n_max=0.1)"]
    C --> E["base_ang_vel noise=Unoise(n_min=-0.2, n_max=0.2)"]
    C --> F["projected_gravity noise=Unoise(n_min=-0.05, n_max=0.05)"]
    C --> G["joint_pos noise=Unoise(n_min=-0.01, n_max=0.01)"]
    C --> H["joint_vel noise=Unoise(n_min=-1.5, n_max=1.5)"]
    C --> I["height_scan noise=Unoise(n_min=-0.1, n_max=0.1)"]

    style B fill:#90EE90
Loading

Reviews (1): Last reviewed commit: "Fix quadruped tutorial noise import" | Re-trigger Greptile

@isaaclab-review-bot isaaclab-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Isaac Lab Review Bot

PR: #6037 — [release/3.0.0-beta2] Fix quadruped tutorial noise import
Author: @ooctipus | Base: release/3.0.0-beta2

Summary

Backport of #6033 to release/3.0.0-beta2. Updates the quadruped tutorial to import UniformNoiseCfg instead of the deprecated AdditiveUniformNoiseCfg.

Review

# Category Finding
No issues found

Details

  • Change: Single-line import fix in scripts/tutorials/03_envs/create_quadruped_base_env.py
  • AdditiveUniformNoiseCfgUniformNoiseCfg (aliased as Unoise)
  • This aligns with the noise API refactor where additive uniform noise was consolidated into UniformNoiseCfg
  • Same commit as #6032 (78e0093d) targeting the beta2 release branch

CI Status

✅ All core checks passing (pre-commit, build wheel, test suites)
⏳ A few infrastructure checks pending (Docker builds, license-check) — unrelated to this change

Verdict

LGTM — Minimal, correct backport fix. No concerns.


Reviewed commit: 78e0093d | 0 findings | [Automated review]

@ooctipus ooctipus merged commit 876fdd8 into isaac-sim:release/3.0.0-beta2 Jun 9, 2026
74 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working isaac-lab Related to Isaac Lab team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants