Skip to content

Commit

Permalink
Add default values to physics shapes and update formatting
Browse files Browse the repository at this point in the history
  • Loading branch information
@aaronfranke
aaronfranke committed Nov 29, 2024
1 parent e7a802c commit e0ae4d3
Show file tree
Hide file tree
Showing 9 changed files with 51 additions and 32 deletions.
Binary file modified extensions/2.0/KHR_audio_emitter/examples/boom_box/textures/boom_box_emission.png
View file
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified ...ions/2.0/OMI_physics_gravity/examples/earth_millionth_scale/earth_roughness.png
View file
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
38 changes: 20 additions & 18 deletions extensions/2.0/OMI_physics_shape/README.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -63,18 +63,18 @@ The extension must also be added to the glTF's `extensionsUsed` array and becaus

The main data structure `glTF.OMI_physics_shape.shape.schema.json` defines a type property.

| | Type | Description | Default value |
| --------------| ----------- | -------------------------------------------------------------------------- | -------------------- |
| **type** | `string` | The type of the physics shape as a string. | Required, no default |
| | Type | Description | Default value |
| -------- | -------- | ------------------------------------------ | -------------------- |
| **type** | `string` | The type of the physics shape as a string. | Required, no default |

In addition to the type, a key with the same name as the type can be used to define a sub-JSON with the details of the shape. Which sub-properties are allowed depend on which shape type is being used. The possible properties are described in the following table.

| | Type | Description | Default value | Valid on |
| --------------| ----------- | -------------------------------------------------------------------------- | -------------------- | ------------------------- |
| **size** | `number[3]` | The size of the box shape in meters. | [1.0, 1.0, 1.0] | Box |
| **radius** | `number` | The radius of the shape in meters. | 0.5 | Sphere, capsule, cylinder |
| **height** | `number` | The height of the shape in meters. | 2.0 | Capsule, cylinder |
| **mesh** | `number` | The index of the glTF mesh in the document to use as a mesh shape. | -1 | Trimesh, convex |
| | Type | Description | Default value | Valid on |
| ---------------- | ----------- | ------------------------------------------------------------------ | --------------- | ------------------------- |
| **size** | `number[3]` | The size of the box shape in meters. | [1.0, 1.0, 1.0] | Box |
| **radius** | `number` | The radius of the shape in meters. | 0.5 | Sphere, capsule, cylinder |
| **height** | `number` | The height of the shape in meters. | 2.0 | Capsule, cylinder |
| **mesh** | `number` | The index of the glTF mesh in the document to use as a mesh shape. | -1 | Trimesh, convex |

If a key for the type is missing, or a sub-JSON key is missing, implementations should use the default value. A mesh index of -1 means invalid.

Expand Down Expand Up @@ -109,7 +109,9 @@ Capsule shapes describe a "pill" shape. They have a `radius` and `height` proper

#### Cylinder

Cylinder shapes describe a "tall circle" shape. They are similar in structure to capsules, they have a `radius` and `height` property. The height is aligned with the node's local vertical axis. If it's desired to align it along a different axis, rotate the glTF node. If the `radius` property is omitted, the default radius is `0.5`, and if the `height` property is omitted, the default height is `2.0`.
Cylinder shapes describe a "tall circle" shape. They are similar in structure to capsules, they have a `radius` and `height` property. If the `radius` property is omitted, the default radius is `0.5`, and if the `height` property is omitted, the default height is `2.0`.

The `height` property of a cylinder describes the total height, the distance between the center of the bottom disc and the center of the top disc. The height is aligned with the node's local vertical axis. If it's desired to align it along a different axis, rotate the glTF node.

The use of cylinder is discouraged if another shape would work well in its place. Cylinders are harder to calculate than boxes, spheres, and capsules. Not all game engines support cylinder shapes. Engines that do not support cylinder shapes should use an approximation, such as a convex hull roughly shaped like a cylinder. Cylinders over twice as tall as they are wide can use another approximation: a convex hull combined with an embedded capsule (to allow for smooth rolling), by copying the cylinder's values into a new capsule shape.

Expand All @@ -131,14 +133,14 @@ Avoid using a trimesh shape for most objects, they are the slowest shapes to cal

The following JSON pointers are defined representing mutable properties defined by this extension, for use with the glTF Object Model including extensions such as `KHR_animation_pointer` and `KHR_interactivity`.

| JSON Pointer | Object Model Type |
| --------------------------------------------------------- | ----------------- |
| `/extensions/OMI_physics_shape/shapes/{}/box/size` | `float3` |
| `/extensions/OMI_physics_shape/shapes/{}/sphere/radius` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/capsule/radius` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/capsule/height` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/cylinder/radius` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/cylinder/height` | `float` |
| JSON Pointer | Object Model Type |
| --------------------------------------------------------------- | ----------------- |
| `/extensions/OMI_physics_shape/shapes/{}/box/size` | `float3` |
| `/extensions/OMI_physics_shape/shapes/{}/sphere/radius` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/capsule/height` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/capsule/radius` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/cylinder/height` | `float` |
| `/extensions/OMI_physics_shape/shapes/{}/cylinder/radius` | `float` |

Additionally, the following JSON pointers are defined for read-only properties:

Expand Down
4 changes: 2 additions & 2 deletions extensions/2.0/OMI_physics_shape/schema/glTF.OMI_physics_shape.schema.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@
"properties": {
"shapes": {
"type": "array",
"description": "An array of shapes that can be referenced by nodes.",
"description": "An array of physics shape resources that can be referenced by nodes.",
"items": {
"type": "object",
"$ref": "shape.schema.json"
"$ref": "glTF.OMI_physics_shape.shape.schema.json"
},
"minItems": 1
},
Expand Down
9 changes: 8 additions & 1 deletion extensions/2.0/OMI_physics_shape/schema/glTF.OMI_physics_shape.shape.box.schema.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,14 @@
"properties": {
"size": {
"type": "array",
"description": "Three number array representing the width, height, and depth of the box (x, y, z)."
"description": "Three number array representing the width, height, and depth of the box (x, y, z).",
"items": {
"type": "number",
"exclusiveMinimum": 0.0
},
"minItems": 3,
"maxItems": 3,
"default": [ 1.0, 1.0, 1.0 ]
},
"extensions": { },
"extras": { }
Expand Down
12 changes: 8 additions & 4 deletions extensions/2.0/OMI_physics_shape/schema/glTF.OMI_physics_shape.shape.capsule.schema.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,17 @@
"description": "Parameters describing a capsule physics shape.",
"allOf": [ { "$ref": "glTFProperty.schema.json" } ],
"properties": {
"radius": {
"height": {
"type": "number",
"description": "The radius to use for the capsule shape."
"description": "The full height of the capsule shape in meters.",
"exclusiveMinimum": 0.0,
"default": 2.0
},
"height": {
"radius": {
"type": "number",
"description": "The full height of the capsule shape."
"description": "The radius of the capsule in meters.",
"minimum": 0.0,
"default": 0.5
},
"extensions": { },
"extras": { }
Expand Down
12 changes: 8 additions & 4 deletions extensions/2.0/OMI_physics_shape/schema/glTF.OMI_physics_shape.shape.cylinder.schema.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,17 @@
"description": "Parameters describing a cylinder physics shape.",
"allOf": [ { "$ref": "glTFProperty.schema.json" } ],
"properties": {
"radius": {
"height": {
"type": "number",
"description": "The radius to use for the cylinder shape."
"description": "The height of the cylinder shape in meters, the distance between the centers of the two capping discs.",
"exclusiveMinimum": 0.0,
"default": 2.0
},
"height": {
"radius": {
"type": "number",
"description": "The height of the cylinder shape."
"description": "The radius of the cylinder in meters.",
"minimum": 0.0,
"default": 0.5
},
"extensions": { },
"extras": { }
Expand Down
4 changes: 2 additions & 2 deletions extensions/2.0/OMI_physics_shape/schema/glTF.OMI_physics_shape.shape.schema.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@
"$id": "glTF.OMI_physics_shape.shape.schema.json",
"title": "OMI_physics_shape Shape Resource",
"type": "object",
"description": "Defines a physics shape resource.",
"description": "Defines a physics shape resource that can be referenced by nodes.",
"allOf": [ { "$ref": "glTFChildOfRootProperty.schema.json" } ],
"properties": {
"type": {
"type": "string",
"description": "The type of shape this node represents.",
"description": "Specifies the shape type of this physics shape.",
"enum": [
"box",
"sphere",
Expand Down
4 changes: 3 additions & 1 deletion extensions/2.0/OMI_physics_shape/schema/glTF.OMI_physics_shape.shape.sphere.schema.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@
"properties": {
"radius": {
"type": "number",
"description": "The radius to use for the sphere shape."
"description": "The radius to use for the sphere shape in meters.",
"minimum": 0.0,
"default": 0.5
},
"extensions": { },
"extras": { }
Expand Down

0 comments on commit e0ae4d3

Please sign in to comment.