fix raycaster visulization errors printed in the terminal

[DreaverZhao]

Description

Fixes #4518 by not calling the visualize function if all ray_hits_w are invalid values. I've also re-sorted the contributors' names alphabetically.

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist

• I have read and understood the contribution guidelines
• I have run the pre-commit checks with ./isaaclab.sh --format
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• I have updated the changelog and the corresponding version in the extension's config/extension.toml file
• I have added my name to the CONTRIBUTORS.md or my name already exists there

[greptile-apps]

Greptile Overview

Greptile Summary

This PR fixes a bug where raycaster visualization would throw errors when all ray hits are invalid (inf values). The fix adds an early return check after filtering infinite values to prevent calling the visualizer with an empty tensor, which would cause a ValueError in VisualizationMarkers.visualize() at line 330 where it requires num_markers > 0. Additionally, the PR re-sorts contributor names alphabetically in CONTRIBUTORS.md as required by project guidelines.

Key Changes:

• Added guard condition in _debug_vis_callback to skip visualization when viz_points.shape[0] == 0
• Moved 5 contributor names (Anke Zhao, David Leon, Song Yi, Tsz Ki GAO, Weihua Zhang) from the end of the list to their correct alphabetical positions

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The fix is a simple defensive check that prevents errors without changing any existing behavior. The logic is sound - it prevents passing an empty tensor to the visualizer which would fail. The CONTRIBUTORS.md changes are purely administrative and follow project guidelines.
• No files require special attention

Important Files Changed

Filename	Overview
source/isaaclab/isaaclab/sensors/ray_caster/ray_caster.py	Added guard to skip visualization when all ray hits are invalid (inf values)
CONTRIBUTORS.md	Re-sorted contributor names alphabetically per project guidelines

Sequence Diagram

sequenceDiagram
    participant Event as Simulation Event
    participant RC as RayCaster
    participant Data as ray_hits_w
    participant VM as VisualizationMarkers
    
    Event->>RC: _debug_vis_callback(event)
    RC->>Data: Check if ray_hits_w is None
    alt ray_hits_w is None
        RC->>Event: return early
    else ray_hits_w exists
        RC->>Data: Filter inf values
        RC->>RC: Check if viz_points.shape[0] == 0
        alt No valid points (NEW FIX)
            RC->>Event: return early (prevents error)
        else Has valid points
            RC->>VM: visualize(viz_points)
            VM->>VM: Update point instances
        end
    end

Loading

[greptile-apps]

_{2 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}