Skip to content

fix: concrete scene builder subclasses not registered - #59

Open
andrewwhitecdw wants to merge 1 commit into
NVlabs:mainfrom
andrewwhitecdw:bugfix/scene-builders-8799b34b
Open

fix: concrete scene builder subclasses not registered#59
andrewwhitecdw wants to merge 1 commit into
NVlabs:mainfrom
andrewwhitecdw:bugfix/scene-builders-8799b34b

Conversation

@andrewwhitecdw

Copy link
Copy Markdown

This PR addresses the following issue in exts/omni.ext.mobility_gen/omni/ext/mobility_gen/scene_builders.py: concrete scene builder subclasses not registered.

Changes

  • exts/omni.ext.mobility_gen/omni/ext/mobility_gen/scene_builders.py: concrete scene builder subclasses not registered.

Details

Before:

class RandomCubeSceneSmall(RandomCubeScene):

After:

@SCENE_BUILDERS.register()
class RandomCubeSceneSmall(RandomCubeScene):

@andrewwhitecdw

Copy link
Copy Markdown
Author

Closing after multiple repair attempts: original patch no longer applies or auditor feedback remains unaddressed.

@andrewwhitecdw
andrewwhitecdw deleted the bugfix/scene-builders-8799b34b branch August 2, 2026 23:58
@andrewwhitecdw
andrewwhitecdw restored the bugfix/scene-builders-8799b34b branch August 17, 2026 21:00
@andrewwhitecdw

Copy link
Copy Markdown
Author

script closed on accident

Signed-off-by: Andrew White <andrewwhitecdw@users.noreply.github.com>
Signed-off-by: andrewwhitecdw <andrewwhitecdw@users.noreply.github.com>
@andrewwhitecdw
andrewwhitecdw force-pushed the bugfix/scene-builders-8799b34b branch from 47c964d to 010e0d5 Compare August 18, 2026 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant