Skip to content

Update the anim state graph to support the new data schema#2278

Merged
ellthompson merged 3 commits intomasterfrom
anim-state-graph-editor-changes
Jul 14, 2020
Merged

Update the anim state graph to support the new data schema#2278
ellthompson merged 3 commits intomasterfrom
anim-state-graph-editor-changes

Conversation

@ellthompson
Copy link
Copy Markdown
Contributor

I confirm I have signed the Contributor License Agreement.

@ellthompson ellthompson marked this pull request as ready for review July 14, 2020 09:06
@ellthompson ellthompson requested review from a team and willeastcott July 14, 2020 09:08
@ellthompson ellthompson self-assigned this Jul 14, 2020
@ellthompson ellthompson added area: animation Animation related issue enhancement labels Jul 14, 2020
Comment thread src/framework/components/anim/state-graph.js
for (var paramId in data.parameters) {
var param = data.parameters[paramId];
this._parameters[param.name] = { type: param.type, value: param.value };
var i;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Might be good to document what data is in this case?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

👍

@ellthompson ellthompson merged commit 887d841 into master Jul 14, 2020
TheJonRobinson added a commit to Mojiworks/playcanvas-engine that referenced this pull request Jul 15, 2020
* mojiworks/pr/sound-stop-typing: (112 commits)
  fix: SoundComponent.stop's string parameter is optional
  [RELEASE] v1.32.0
  [Fix] Changes to Attribute to location mapping (playcanvas#2292)
  Deprecate pc.makeArray (playcanvas#2289)
  [FIX] API ref docs for pc.Mesh#getIndices (playcanvas#2291)
  fix cubemap filtering on safari (playcanvas#2288)
  disable ImageBitmap (playcanvas#2286)
  [Fix] Fix to texture based skinning on Safari (playcanvas#2287)
  fix cubemap dependent loads (playcanvas#2285)
  [Fix] Update to Transform Feedback to work with VAOs (playcanvas#2284)
  Update the anim state graph to support the new data schema (playcanvas#2278)
  Remove viewer (playcanvas#2282)
  [FIX] pc.SoundComponent#resume/stop param type (playcanvas#2281)
  Remove type function (playcanvas#2273)
  Remove some shader-related circular dependencies (playcanvas#2274)
  Glb-parser improvements (playcanvas#2275)
  Fix glb-parser normals generation (playcanvas#2276)
  Glb texture coordinate fix (playcanvas#2252)
  use faces if the asset was previously unloaded (playcanvas#2272)
  fix material resource loader (playcanvas#2268)
  ...

# Conflicts:
#	src/scene/graph-node.js
@willeastcott willeastcott deleted the anim-state-graph-editor-changes branch January 12, 2021 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: animation Animation related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants