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.

Sign up for GitHub

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

Jump to bottom

EXT_animation_map extension proposal #1137

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

EXT_animation_map extension proposal #1137

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
730df4e
Add EXT_animation_map extension spec
msfeldstein Oct 26, 2017
2bb65b0
EXT -> FB
msfeldstein Oct 26, 2017
6a022ac
Clarify animation play type
msfeldstein Oct 31, 2017
5cf7dba
serial->sequential
msfeldstein Oct 31, 2017
09ea0c8
Specify Nodes for interactive animations
msfeldstein Oct 31, 2017
cfa2bf7
node->nodes, specify that any node can trigger
msfeldstein Oct 31, 2017
24901c3
Update to specify that it can be any nodes triggering
msfeldstein Oct 31, 2017
d7a5a25
number --> integer
msfeldstein Oct 31, 2017
04f7005
Add unique items and min items
msfeldstein Oct 31, 2017
211dd1d
Use EXT_ instead of FB_
msfeldstein Dec 16, 2017
087a563
Add pointer semantics
msfeldstein Dec 16, 2017
800e38d
Update README.md
msfeldstein Dec 22, 2017
551b19d
Update the semantics/nodes binding wording
Jul 9, 2018
bb1e9fb
Update the semantics/nodes binding wording
Jul 9, 2018
d7761f8
formalize semantic names
Jul 9, 2018
955ec8b
Add glTF property schemas to animation map schema
Jul 10, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
node->nodes, specify that any node can trigger
  • Loading branch information
@msfeldstein
msfeldstein committed Oct 31, 2017
commit cfa2bf76b0bb28b9d80bc180f1f60082ccf87e36
6 changes: 2 additions & 4 deletions extensions/Vendor/FB_animation_map/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ It maps a semantic, which can be any string a client wishes to bind, to an array
* **LEAVE**: Trigger a single shot animation as the object is leaving the scene or viewport
* **ALWAYS**: Constantly loop an animation

The following semantics require a list of glTF Node objects to specify interaction targets.
The following semantics require a list of glTF Node objects to specify interaction targets. If the nodes are omitted, it can be assumed that the entire glTF Scene is the target.

* **TRIGGER**: A user triggering a single shot animation on Touch, Tap, Click, or any other way of activating
* **GRAB**: Triggered when an object is picked up, for example grabbed in vr, or dragged in WebGL.
Expand All @@ -41,8 +41,6 @@ The following semantics require a list of glTF Node objects to specify interacti
* **PROXIMITY**: Looped while a viewer is near to an object
* **PROXIMITY_LEAVE**: Triggered when a viewer gets further from an object

The 'object' referred to is the mesh

## glTF Schema Updates

The new FB_animation_map property is a root glTF level optional extension object. The **bindings** array is a list of objects containing a semantic, an array of animations mapped to that semantic, and an optional list of nodes that this animation targets. The node parameter is used for interactive animations like GRAB or GAZE. Animations should be played all at once, though future additions could add parallel (default), sequential, or random play modes.
Expand Down Expand Up @@ -72,7 +70,7 @@ The new FB_animation_map property is a root glTF level optional extension object
{
"semantic": "GAZE_ENTER",
"animations": [4],
"node": [3]
"nodes": [3]
}
]
}
Expand Down
10 changes: 9 additions & 1 deletion extensions/Vendor/FB_animation_map/schema/FB_animation_map.schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,16 @@
"type": "number",
Copy link
Member

Choose a reason for hiding this comment

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

number -> integer

"minimum": 0
}
},
"nodes": {
"type": "array",
"items": {
"type": "number",
Copy link
Member

Choose a reason for hiding this comment

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

number -> integer

"minimum": 0
}
}
}
},
"required": ["semantic", "animations"]
}
},
"properties" : {
Expand Down