Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace OMI_collider with OMI_physics_shape #171

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

aaronfranke
Copy link
Contributor

@aaronfranke aaronfranke commented Jun 9, 2023

This is based on the discussions that took place in the 2023-06-07 Khronos 3D formats IP-free physics meeting and the 2023-06-08 OMI glTF group meeting. There was a desire to separate the concerns of defining the shape from its behavior such as static, kinematic, rigid, trigger, etc. This means taking OMI_collider, removing the independent behavior, removing isTrigger, and renaming it to OMI_physics_shape.

These physics shapes could also be used by other extensions if desired, since all they do is define a shape compatible with a physics engine.

This breaks compatibility with all implementations, therefore automatically demoting the specs to Stage 1.

Copy link
Contributor

@antpb antpb left a comment

Choose a reason for hiding this comment

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

From a formatting and code structure perspective this lgtm 👍

@antpb
Copy link
Contributor

antpb commented Jun 22, 2023

Screenshot 2023-06-22 at 5 03 07 PM Screenshot 2023-06-22 at 5 03 46 PM

Vote passes with 7 in favor none in opposition.

@antpb antpb merged commit 4025586 into omigroup:main Jun 22, 2023
@aaronfranke aaronfranke deleted the OMI_physics_shape branch June 22, 2023 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants