fix: Auto-detect SH degree for Isaac Sim NuRec compatibility #147
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.
Summary
Fixes critical performance degradation when rendering 3DGRUT-exported USDZ files in Isaac Sim. The issue was caused by parameter mismatch between
n_active_featuresandradiance_sph_degreein NuRec configuration.Problem
f_rest_*attributes) export withn_active_features=0radiance_sph_degreeremained at config default (typically 3)Solution
Automatic SH Degree Detection in Model Layer
model.py: Auto-detect SH degree from PLY file structure ininit_from_ply()usdz_exporter.py: Ensureradiance_sph_degreematchesn_active_featuresply_to_usd.py: Simplified script leveraging model-layer intelligenceKey Benefits
Testing
Files Changed
threedgrut/model/model.py: Auto SH detection ininit_from_ply()threedgrut/export/usdz_exporter.py: Parameter consistency enforcementthreedgrut/export/scripts/ply_to_usd.py: Simplified logicImpact
This PR makes 3DGRUT exports truly usable in Isaac Sim for the first time! 🚀