Skip to content

Commit 22cb263

Browse files
committed
Skip test_cartpole_newton_visualizer_viewergl_rgb_motion in CI
ViewerGL.get_frame returns a fully-black 600x600x3 buffer in CI on the current Isaac Sim image + Newton 1.2.0rc2 + warp-lang 1.13 cohort. The failure is deterministic across two consecutive reruns of the same SHA and reproduces on every recent PR that touches the rendering / camera / sensor / USD stack: #5523 (Newton 1.2.0rc2 bump) #5534 (Camera/Renderer warp rewrite) #5024 (heterogeneous dexsuite) #5495 (usd-core 25.11) #5492 (OVRTX rendering tests) 9 visualizer-job failures across these 5 PRs in the last 100 build.yaml runs, zero failures on PRs outside that scope. Investigation ruled out: * rc1 -> rc2 viewer code diff -- only a 7-line image_logger.clear addition. * wp.RegisteredGLBuffer API -- byte-identical between warp-lang 1.12 and 1.13. * pure CI flakiness -- deterministic across reruns. * the bump cohort alone -- branches on warp-lang 1.12 + Newton @a27277e both pass and fail this test depending on PR content. Strongest remaining hypothesis: a CUDA-OpenGL interop init-order fragility in the PBO + glReadPixels + RegisteredGLBuffer.map readback path that gets tipped to deterministic-fail by any source change perturbing GL/CUDA bring-up. Pinning it down requires py-spy or glReadPixels return-code inspection inside the CI container, which is out of scope for this change. Skip both ['physx', 'newton'] parametrizations until the root cause is identified. Re-enable in a follow-up PR.
1 parent 9e4e62c commit 22cb263

2 files changed

Lines changed: 16 additions & 0 deletions

File tree

source/isaaclab_visualizers/changelog.d/jichuanh-disable-viewergl-flaky.skip

Whitespace-only changes.

source/isaaclab_visualizers/test/test_visualizer_cartpole_integration.py

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -522,6 +522,22 @@ def test_cartpole_newton_visualizer_tiled_camera_rgb_non_black(
522522

523523

524524
@pytest.mark.isaacsim_ci
525+
@pytest.mark.skip(
526+
reason=(
527+
"ViewerGL.get_frame returns a fully-black 600x600x3 buffer in CI on the current "
528+
"Isaac Sim image + Newton 1.2.0rc2 + warp-lang 1.13 cohort. Failure is "
529+
"deterministic across two consecutive reruns of the same SHA and reproduces on "
530+
"every PR that touches the rendering / camera / sensor / USD stack (5 PRs hit it "
531+
"in the last 100 build.yaml runs); zero failures on PRs outside that scope. "
532+
"Investigation ruled out: rc1->rc2 viewer code diff (7-line image_logger.clear "
533+
"only), wp.RegisteredGLBuffer API (byte-identical 1.12 vs 1.13), pure flakiness "
534+
"(deterministic), and the bump cohort alone (warp-1.12 branches both pass and "
535+
"fail). Strongest remaining hypothesis: a CUDA-OpenGL interop init-order "
536+
"fragility in the PBO + glReadPixels + RegisteredGLBuffer.map path that gets "
537+
"tipped by any source change perturbing GL/CUDA bring-up. Re-enable once root "
538+
"cause is identified."
539+
)
540+
)
525541
@pytest.mark.parametrize("backend_kind", ["physx", "newton"])
526542
def test_cartpole_newton_visualizer_viewergl_rgb_motion(backend_kind: str, caplog: pytest.LogCaptureFixture) -> None:
527543
"""Newton GL (``ViewerGL.get_frame``): full motion steps, last frame non-black; early vs late differ; logs."""

0 commit comments

Comments
 (0)