Commit Graph

66 Commits

Author SHA1 Message Date
Jurre deBaare
ff0db16d0b Reintroducing AnimationDataModel refactor with additional memory and cooktime optimizations.
**Animation Data Model load/cook time and memory optimizations**
- Added SetKeysOnly to FMovieSceneFloatChannel, allowing to only set value/frames values and skip allocating key handles
- Added EvaluateWithCache to TMovieSceneCurveChannelImpl, allowing to evaluate multiple curves using a caching data structure holding the frame-indices resulting from the FrameTime -> indexes calculationg
- UFKControlRig::GetControlName/GetControlTargetName, added thread_local mapping cache and optimized FName generation
- Made the sequencer bone curve keys 'sparse' again, if constant it just sets the default value for the curve otherwise it is still uniform
- Only resample keys when compressing, and clear them on any significant model changes and when save-during-cook has been completed
- Routed ::Presave for AnimSequencerDataModel directly through to UMovieSceneSignedObject to skip compiling cooked sequencer that that will/should never cooked and packaged
- Added some significant pass-by-reference changes to AnimSequencerModel APIs (passing large arrays by value previously) :doh:
- Optimized GenerateLegacyBoneData

Original CL description:

**New**
- Engine
	- IAnimationDataModel interface
- AnimationData plugin
	- AnimSequencerDataModel, sequencer based implementation of IAnimationDataModel
	- AnimSequencerDataController, controller for above (implementation of IAnimationDataController)
- Added FCompressedRichCurve::PopulateCurve, allowing users to convert back to a FRichCurve for validation/debugging
- Added DefaultFrameRate to AnimationSettings, this replaces the hardcoded 30FPS in code
- Added ::GetKeyIndicesFromTime which takes FFrameRate
- Added AnimSequenceBase::OnAnimModelLoaded, this is required for correct postloading behaviour of data model (specifically AnimSequencerDataModel, as it depends on data from its outer ? anim sequence)
- Added IAnimationDataModel ::Evaluate which now takes responsibility for evaluating raw animation bone, curve and attribute data. And moved all trackbased evaluation code into AnimDataModel.cpp
- Added a.ForceEvalRawData allowing to force evaluation of source data
- Added a.SkipDDC to force compressing animation data locally

**Changed**
- Reparent UAnimDataModel to IAnimationDataModel interface, and rejig behaviour
- AnimSequenceBase now reference AnimDataModel as IAnimationDataModel
- Upgrade path for legacy to IAnimationDataModel and existing UAnimDataModel to IAnimationDataModel through IAnimationDataController::PopulateWithExisting
- IAnimationDataModel data is now frame(number/rate/time) based rather than seconds/keyindices. This enforces frameborder aligned data from now on.
- Moved RichCurve evaluation code from RichCurve.cpp to separate file and consolidated all instances of copy-pasta behaviour
- Updated/removed deprecated EngineTests around AnimSequences
- Changed many instances to use a FFrameTime together with a known FFrameRate versus seconds and sequencelength in float involving frame \/ time calculations (including instances of FAnimKeyHelper)
- Switched anim sequence evaluation to use double in Extraction context for time value and patched up places with static\_cast\<double\>
- FRichCurve::Eval
	- Make sure we always evaluate keys when T >= Key0.Time and T <= KeyN.Time to deal with crazy userweighted tangents
- Fixed WeightedKeyDataAdapter::GetKeyInterpMode/GetKeyTangentWeightMode retrieving invalid values, as it was indexing according to KeyIndex rather than KeyIndex\*2. This made it so that values were misinterpreted.
- Fixed WeightedEvalForTwoKeys for compressed data retrieving GetKeyInterpMode and GetKeyTangentWeightMode using the wrong index value. As they are not indexed using KeyDataHandle type.
- Fixed issue where compression could crash when containing zero-key scale additivebase track (tries to retrieve)
- Replaced instances of NULL with nullptr
- Moved required decompression information into FAnimSequenceDecompressionContext and reimplemented decompression within UE::Anim::Decompression namespace and new file
- Deprecated ::GetRawDataGuid and replaced with GetDataModel()->GenerateGuid()
- Fixed UAnimStreamable evaluation/compression (previously broken with MVC refactor)
- Updated Animation exporting pipeline to be frame-based rather than seconds
- Deprecated RetargetPose, replaced with FRetargetingScope (used within AnimModel::Evaluate)
- Fixed BaseAdditiveAnimation array become invalid/incorrect when removing zero-additve tracks during compression
- Updated EngineTest animation sequence related tests to new APIs (while maintaining deprecated path testing as well for now)
- AnimDataController / Animation Sequence tests now generate a USkeleton for the transient animation sequence (used to perform the test on)

**Removed**
- Unused file/class AnimData/AnimDataNotifyCollector.h

#rb Thomas.Sarkanen, Mike.Zyracki
#jira UE-131296
#preflight 631f6897a9331c69c3a34d1e

[CL 21980597 by Jurre deBaare in ue5-main branch]
2022-09-13 06:41:15 -04:00
kriss gossart
0422ca7705 Skeletal Mesh - Replace the newly created GetSkeletalMesh function by GetSkeletalMeshAsset so it matches the setter SetSkeletalMeshAsset function (which itself couldn't be named SetSkeletalMesh due to the function already existing and doing something else).
#rb Josie.Yang
#preflight 62fa2afeae3edb54c979492e
#jira none

[CL 21385959 by kriss gossart in ue5-main branch]
2022-08-15 09:26:50 -04:00
Josie Yang
6b15506e58 Replace direct access to SkeletalMesh object from USkinnedMeshComponent with GetSkeletalMesh function
#rb kriss.gossart
#preflight 62aafc9ada0af39a4783930a

[CL 20686007 by Josie Yang in ue5-main branch]
2022-06-16 09:14:04 -04:00
Marc Audy
de2c3cb8e0 [Backout] - CL20593017, 20594056, 20625634, 20645670, 20649179, 20649223, 20649380, 20658535
Add new custom version to specify a certain window of AnimSequences are not loadable
#fyi Jurre.deBaare
#robomerge EngineMerge
Original CL Desc
-----------------------------------------------------------------
**New**
- Engine
	- IAnimationDataModel interface
- AnimationData plugin
	- AnimSequencerDataModel, sequencer based implementation of IAnimationDataModel
	- AnimSequencerDataController, controller for above (implementation of IAnimationDataController)
- Added FCompressedRichCurve::PopulateCurve, allowing users to convert back to a FRichCurve for validation/debugging
- Added DefaultFrameRate to AnimationSettings, this replaces the hardcoded 30FPS in code
- Added ::GetKeyIndicesFromTime which takes FFrameRate
- Added AnimSequenceBase::OnAnimModelLoaded, this is required for correct postloading behaviour of data model (specifically AnimSequencerDataModel, as it depends on data from its outer ? anim sequence)
- Added IAnimationDataModel ::Evaluate which now takes responsibility for evaluating raw animation bone, curve and attribute data. And moved all trackbased evaluation code into AnimDataModel.cpp
- Added a.ForceEvalRawData allowing to force evaluation of source data
- Added a.SkipDDC to force compressing animation data locally

**Changed**
- Reparent UAnimDataModel to IAnimationDataModel interface, and rejig behaviour
- AnimSequenceBase now reference AnimDataModel as IAnimationDataModel
- Upgrade path for legacy to IAnimationDataModel and existing UAnimDataModel to IAnimationDataModel through IAnimationDataController::PopulateWithExisting
- IAnimationDataModel data is now frame(number/rate/time) based rather than seconds/keyindices. This enforces frameborder aligned data from now on.
- Moved RichCurve evaluation code from RichCurve.cpp to separate file and consolidated all instances of copy-pasta behaviour
- Updated/removed deprecated EngineTests around AnimSequences
- Changed many instances to use a FFrameTime together with a known FFrameRate versus seconds and sequencelength in float involving frame \/ time calculations (including instances of FAnimKeyHelper)
- Switched anim sequence evaluation to use double in Extraction context for time value and patched up places with static\_cast\<double\>
- FRichCurve::Eval
	- Make sure we always evaluate keys when T >= Key0.Time and T <= KeyN.Time to deal with crazy userweighted tangents
- Fixed WeightedKeyDataAdapter::GetKeyInterpMode/GetKeyTangentWeightMode retrieving invalid values, as it was indexing according to KeyIndex rather than KeyIndex\*2. This made it so that values were misinterpreted.
- Fixed WeightedEvalForTwoKeys for compressed data retrieving GetKeyInterpMode and GetKeyTangentWeightMode using the wrong index value. As they are not indexed using KeyDataHandle type.
- Fixed issue where compression could crash when containing zero-key scale additivebase track (tries to retrieve)
- Replaced instances of NULL with nullptr
- Moved required decompression information into FAnimSequenceDecompressionContext and reimplemented decompression within UE::Anim::Decompression namespace and new file
- Deprecated ::GetRawDataGuid and replaced with GetDataModel()->GenerateGuid()
- Fixed UAnimStreamable evaluation/compression (previously broken with MVC refactor)
- Updated Animation exporting pipeline to be frame-based rather than seconds
- Deprecated RetargetPose, replaced with FRetargetingScope (used within AnimModel::Evaluate)
- Fixed BaseAdditiveAnimation array become invalid/incorrect when removing zero-additve tracks during compression
- Updated EngineTest animation sequence related tests to new APIs (while maintaining deprecated path testing as well for now)
- AnimDataController / Animation Sequence tests now generate a USkeleton for the transient animation sequence (used to perform the test on)

**Removed**
- Unused file/class AnimData/AnimDataNotifyCollector.h

#rb Thomas.Sarkanen, Martin.Wilson, Alexis.Matte, Mike.Zyracki
#jira UE-131296
#preflight 62a308a8b0150a87f9d6891b

[CL 20677979 by Marc Audy in ue5-main branch]
2022-06-15 18:24:46 -04:00
Jurre deBaare
e13fdba72c **New**
- Engine
	- IAnimationDataModel interface
- AnimationData plugin
	- AnimSequencerDataModel, sequencer based implementation of IAnimationDataModel
	- AnimSequencerDataController, controller for above (implementation of IAnimationDataController)
- Added FCompressedRichCurve::PopulateCurve, allowing users to convert back to a FRichCurve for validation/debugging
- Added DefaultFrameRate to AnimationSettings, this replaces the hardcoded 30FPS in code
- Added ::GetKeyIndicesFromTime which takes FFrameRate
- Added AnimSequenceBase::OnAnimModelLoaded, this is required for correct postloading behaviour of data model (specifically AnimSequencerDataModel, as it depends on data from its outer ? anim sequence)
- Added IAnimationDataModel ::Evaluate which now takes responsibility for evaluating raw animation bone, curve and attribute data. And moved all trackbased evaluation code into AnimDataModel.cpp
- Added a.ForceEvalRawData allowing to force evaluation of source data
- Added a.SkipDDC to force compressing animation data locally

**Changed**
- Reparent UAnimDataModel to IAnimationDataModel interface, and rejig behaviour
- AnimSequenceBase now reference AnimDataModel as IAnimationDataModel
- Upgrade path for legacy to IAnimationDataModel and existing UAnimDataModel to IAnimationDataModel through IAnimationDataController::PopulateWithExisting
- IAnimationDataModel data is now frame(number/rate/time) based rather than seconds/keyindices. This enforces frameborder aligned data from now on.
- Moved RichCurve evaluation code from RichCurve.cpp to separate file and consolidated all instances of copy-pasta behaviour
- Updated/removed deprecated EngineTests around AnimSequences
- Changed many instances to use a FFrameTime together with a known FFrameRate versus seconds and sequencelength in float involving frame \/ time calculations (including instances of FAnimKeyHelper)
- Switched anim sequence evaluation to use double in Extraction context for time value and patched up places with static\_cast\<double\>
- FRichCurve::Eval
	- Make sure we always evaluate keys when T >= Key0.Time and T <= KeyN.Time to deal with crazy userweighted tangents
- Fixed WeightedKeyDataAdapter::GetKeyInterpMode/GetKeyTangentWeightMode retrieving invalid values, as it was indexing according to KeyIndex rather than KeyIndex\*2. This made it so that values were misinterpreted.
- Fixed WeightedEvalForTwoKeys for compressed data retrieving GetKeyInterpMode and GetKeyTangentWeightMode using the wrong index value. As they are not indexed using KeyDataHandle type.
- Fixed issue where compression could crash when containing zero-key scale additivebase track (tries to retrieve)
- Replaced instances of NULL with nullptr
- Moved required decompression information into FAnimSequenceDecompressionContext and reimplemented decompression within UE::Anim::Decompression namespace and new file
- Deprecated ::GetRawDataGuid and replaced with GetDataModel()->GenerateGuid()
- Fixed UAnimStreamable evaluation/compression (previously broken with MVC refactor)
- Updated Animation exporting pipeline to be frame-based rather than seconds
- Deprecated RetargetPose, replaced with FRetargetingScope (used within AnimModel::Evaluate)
- Fixed BaseAdditiveAnimation array become invalid/incorrect when removing zero-additve tracks during compression
- Updated EngineTest animation sequence related tests to new APIs (while maintaining deprecated path testing as well for now)
- AnimDataController / Animation Sequence tests now generate a USkeleton for the transient animation sequence (used to perform the test on)

**Removed**
- Unused file/class AnimData/AnimDataNotifyCollector.h

#rb Thomas.Sarkanen, Martin.Wilson, Alexis.Matte, Mike.Zyracki
#jira UE-131296
#preflight 62a308a8b0150a87f9d6891b

[CL 20593017 by Jurre deBaare in ue5-main branch]
2022-06-10 06:24:32 -04:00
jurre debaare
524ceab54b Non-looping Montage animations pop and don't finish their playback duration in previewer
#fix ensure that blending (in/out) multiplier for montage instance is zero-ed (matching old behaviour)
#rb Thomas.Sarkanen
#preflight 62333348ef5d0f0e8f03a13e

#ROBOMERGE-AUTHOR: jurre.debaare
#ROBOMERGE-SOURCE: CL 19421374 via CL 19421402 via CL 19422905 via CL 19429327 via CL 19429393
#ROBOMERGE-BOT: UE5 (Release-Engine-Staging -> Main) (v930-19419903)

[CL 19432575 by jurre debaare in ue5-main branch]
2022-03-18 02:34:06 -04:00
helge mathee
645e1c7469 Control Rig: Support modifybone to work without a preview mesh
#rb sara.schvartzman
#jira UE-141661
#preflight https://horde.devtools.epicgames.com/job/62027c51bf1c496911476ca7
#lockdown juan.canada

#ROBOMERGE-AUTHOR: helge.mathee
#ROBOMERGE-SOURCE: CL 18902011 in //UE5/Release-5.0/... via CL 18902196 via CL 18902397
#ROBOMERGE-BOT: UE5 (Release-Engine-Test -> Main) (v912-18901109)

[CL 18902410 by helge mathee in ue5-main branch]
2022-02-08 10:30:46 -05:00
aurel cordonnier
34f55d3a4a Merge from Release-Engine-Test @ 17946149 to UE5/Main
This represents UE4/Main @17911760, Release-5.0 @17915875 and Dev-PerfTest @17914035

[CL 17949667 by aurel cordonnier in ue5-main branch]
2021-10-27 15:14:40 -04:00
timothy daoust
aab28b4589 Pausing animation sequences with a single frame
#jira UE-86940
#rb thomas.sarkanen

#ROBOMERGE-SOURCE: CL 15340963 in //UE5/Release-5.0-EarlyAccess/...
#ROBOMERGE-BOT: STARSHIP (Release-5.0-EarlyAccess -> Main) (v771-15082668)

[CL 15340972 by timothy daoust in ue5-main branch]
2021-02-05 15:36:32 -04:00
Thomas Sarkanen
6ebd594d49 Fixed crash opening a pose asset
The pose asset code path in the preview instance used to run its graph udpate in UpdateAnimation(), which meant that the regular scoped sync wasnt applied. Switching it to run in UpdateAnimationNode() means that everyting functions as normal.

#jira UE-107354 - Crash when attempting to open a pose asset
#rb jurre.debaare

[CL 15238297 by Thomas Sarkanen in ue5-main branch]
2021-01-28 10:55:26 -04:00
Jurre deBaare
f24003f082 Animation data MVC refactor
#jira UE-104234
#rb Thomas.Sarkanen, Martin.Wilson, Alexis.Matte, Michael.Zyracki

+ Introduced UAnimDataModel, this currently represents the source data for bone and curve animation. It contains:
    + Bone animation tracks (FBoneAnimationTrack)
        + Key data (coarse)
        + Name
        + Bone tree index
    + Curve data (FAnimationCurveData)
        + Float Curves
        + Transform Curves
    + Play length
    + Sampling rate
        + Used for deriving the expected number of keys/frames when combined with the PlayLength
    + Transient data for supporting backwards compatibility APIs
    + (Dynamic) delegate which broadcasts the mutation Notifies

+ Introduced UAnimDataController, this has sole authority over mutating data contained by UAnimDataModel
    + API functionality allows to transform the contained data in all ways currently expected in the engine.
        + Any mutation will add an undo/redo-able FChange object into the transaction buffer
            + FChangeTransactor helper object allows for keeping track of (compounded) FChange's and inserting them into GUndo
        + Broadcasts change notifies alongside a payload object containing details about the change
            + Interested systems/objects can register to these changes through UAnimDataModel::OnModelModified event
        + Almost all functionality is exposed to both Blueprint and Python scripting
    + Allows for opening 'Brackets', these define the beginning and end of a chain of mutations. Allowing anything registered to OnModelModified to halt responding to the notifies until the (top-level) bracket is closed
    + Undo/redo actions
        + Each mutation to the UAnimDataModel is covered by a 'Action' which is based of FChange and is used to insert a undo/redo-able operation into the transactions buffer

+ Introduced EAnimDataModelNotifType and per-notify payload types. These are used to update views, and any model derived data

+ Introduced FAnimationCurveIdentifier, exposed to scripting, this is used for referencing a curve by name and type when passed to the controller
    + Allows for targetting a specific RichCurve as part of a TransformCurve

+ Introduced FAnimationDataNotifyCollectorused for keeping track of which notifies of type EAnimDataModelNotifType are broadcasted between top-level EAnimDataModelNotifType::BracketOpened and EAnimDataModelNotifType::BracketClosed notifies

+ Added CanTransact to UEngine, returns whether or not a transaction can be made

+ Added UAnimCompositeBase::SetCompositeLength, used for runtime changing of play length value

+ Animation Sequence helpers, containing 'helper' functionality for both AnimSequence and AnimDataModel

* Animation Sequence Base
    + virtual PopulateModel, called during upgrade path for converting existing (legacy) data to the new UAnimDataModel data
    + virtual OnModelModified, registered to the Model's OnModified event to handle any Notifies and payloads
    + Added UAnimDataModel sub-object (editor only)
    + Added UAnimDataController instance (transient and editor only)
    + FAnimationDataNotifyCollector to keep track of model bracketed notifies
    * Deprecated
        * SetSequenceLength()
        * RefreshCurveData()
        * MarkRawDataAsModified()
        * RegisterOnAnimCurvesChanged()
        * UnregisterOnAnimCurvesChanged()
        * UnregisterOnAnimTrackCurvesChanged()
        * RegisterOnAnimTrackCurvesChanged()
        * Public access to RawCurveData

* Animation Sequence
    + Implements PopulateModel/OnModelModified for AnimationSequence related data
    + Added TargetFrameRate, currently locked to the initial sampling rate, but allows for resampling the UAnimDataModel data according to the set rate
    + Added NumberOfSampledKeys (editor only), populated by resampling process
    + Added NumberOfSampledFrames (editor only), populated by resampling process
    + Added ResampledAnimationTrackData (editor only and transient), populated by resampling process
    + Added bBlockCompressionRequests (editor only), used for blocking compression during automation tests
    * Deprecated
        * SetNumberOfSampledKeys()
        * MarkRawDataAsModified()
        * GetRawAnimationData()
        * GetAnimationTrackNames()
        * AddNewRawTrack()
        * GetRawTrackToSkeletonMapTable()
        * GetRawAnimationTrack()
        * GetRawAnimationTrack()
        * UpdateFrameRate()
        * ExtractBoneTransform()
        * CompressRawAnimData()
        * GetSkeletonIndexFromRawDataTrackIndex()
        * RecycleAnimSequence()
        * CleanAnimSequenceForImport()
        * CopyNotifies()
        * PostProcessSequence()
        * AddLoopingInterpolation()
        * RemoveAllTracks()
        * DoesContainTransformCurves()
        * InsertFramesToRawAnimData()
        * CropRawAnimData()
        * RemoveTrack()
        * InsertTrack()
        * ResizeSequence()
        * Non-editor access to GetUncompressedRawSize()
        * Non-editor access to GetApproxRawSize()
        * Public access to UpdateCompressedCurveName()
        * NumberOfKeys
        * SamplingFrameRate
        * TrackToSkeletonMapTable
        * RawAnimationData
        * AnimationTrackNames

* Animation Streamable
    * Model from source Sequence is duplicated rather than copying animation data

* Deprecated ERawCurveTrackTypes::RCT_Vector (marked as hidden)

+ Added automated test for
    + All script exposed controller functionality
    + Undo/redo actions
* Updated existing AnimSequence tests with deprecation and new expected data

+ Base data for compression is now populated from the 'resampled' version stored on the AnimSequence
+ Data cleanup and validation is now performed during compression
    + Track sanitization is now performed during compression (normalizing Quats and clamping near-zero values to zero)
    + Key reduction
    + Invalid track (missing bone from skeleton) removal
    + Curve name validation against skeleton
* Replaced RawCurves with Float curves
* Updated DDC key
- Removed all Guid generation related functionality (now part of the model)

* AnimSequenceBase model registers to AnimModel's notify event, used for refreshing the tracks
    * Replaces the in-place calling of RefreshTracks()
* All curve tracks now hold a const CurveType* rather than a CurveType*/& for introspection of its data
    + Any mutations now go through the controller API
    + Uses AnimationCurveIdentifier to set Transform's per-channel tracks
* RichCurveEditorModelNamed conformed to model/controller
    + Any modifications are applied to an temporary CurveType which always contains a copy of the const-ptr after any previous mutation
    + Registers to the outer AnimModel's notify event, used for updating the cached curve data
    + Registers to CurveModifiedDelegate, used for copying the temp curve data to the model using SetCurveKeys
        * Not-ideal but point that I got to for V1
        * Would be replaced with either refactoring FRichCurve to be MVC like, or by calling UAnimDataController functionality from an implementation of FRichCurveEditorModel

* Mark FChange overrides as final for swap/command change permutations
* Changed FCompoundChange GetDescription to return concatenation of all sub-changes
* Added GetDescription to FTransaction which returns the ToString() value of any contained FChange entries.
    * Entries in SUndoHistory tab now have a ToolTip showing the GetDescription() value
+ FChangeTransactor helper object allows for keeping track of (compounded) FChange's and inserting them into GUndo

* Deprecation handling, conforming code to new APIs and exposing structure/objects/properties to scripting
* Conforming AnimSequence importing from FBX to Model/Controller changes

[CL 15106211 by Jurre deBaare in ue5-main branch]
2021-01-15 06:41:11 -04:00
timothy daoust
3b3bf723c4 Pause the animation timeline for animation sequences with 1 frame
#jira UE-86940  (Pause or Lock AnimTimeline for Animation Sequences with only 1 frame when opened)

#rb thomas.sarkanen

[CL 14879724 by timothy daoust in ue5-main branch]
2020-12-08 12:53:04 -04:00
Jurre deBaare
6f3438b57a UAnimSequence:
- Deprecated NumFrames instead added NumberOfKeys (data is copied over during serialization patch up)
- Deprecated GetNumberOfFrames and GetRawNumberOfFrames instead added GetNumberOfSampledKeys
- Deprecated SetRawNumberOfFrames and instead added SetNumberOfSampledKeys
- Added SamplingFrameRate (FFrameRate), which is populated according to the sequence's actual number of frames and length
- Changed some behaviour to use FFrameRate API to calculate frame number / timings
- Deprecated GetFrameRate and instead added GetSamplingFrameRate
- Fixed uses where NumFrames was assumed to contain the number of frames rather than the number of keys (ResizeSequence, AddLoopingInterpolation, CropAnimation, InsertKeysIntoRawData)

UAnimStreamable:
- Deprecated NumFrames instead added NumberOfKeys (data is copied over during serialization patch up)

ITimeManagementModule:
- Exposed GetCommonFrameRates through the abstract interface to get around circular Engine module dependency

#jira UE-102191
#rb Martin.Wilson

[CL 14608938 by Jurre deBaare in ue5-main branch]
2020-10-29 09:47:22 -04:00
Jurre deBaare
ea7a43d36e Momentum: Deprecating public access to SequenceLength
+ Added SetSequenceLength
    + Moved GetPlayLength virtaul from AnimSequenceBase to AnimationAsset and deprecated GetMaxCurrentTime (only used in a single place)
#rb Martin.Wilson

[CL 14298631 by Jurre deBaare in ue5-main branch]
2020-09-11 10:31:52 -04:00
Lina Halper
44c1bb3292 COPY from //Dev-Anim to //Dev-Main
#rb: none
#fyi: Laurent.Delayen, Thomas.Sarkanen

[CL 11088765 by Lina Halper in Main branch]
2020-01-22 17:58:55 -05:00
ryan durand
627baf970a Updating copyright for Engine Editor.
#rnx
#rb none


#ROBOMERGE-SOURCE: CL 10869241 via CL 10869527 via CL 10869904
#ROBOMERGE-BOT: (v613-10869866)

[CL 10870586 by ryan durand in Main branch]
2019-12-26 15:33:43 -05:00
chris caulfield
5ca6ab385e Chaos - hack in Poke force in PhAt
#rb none


#ROBOMERGE-SOURCE: CL 10837680 via CL 10841173 via CL 10843458 via CL 10843562
#ROBOMERGE-BOT: (v610-10636431)

[CL 10843725 by chris caulfield in Main branch]
2019-12-19 23:33:50 -05:00
Thomas Sarkanen
fdcba8f624 Merging //UE4/Dev-Main to Dev-Anim (//UE4/Dev-Anim) @ CL 4822026
#rb none
#jira none

[CL 4822205 by Thomas Sarkanen in Dev-Anim branch]
2019-01-28 05:55:22 -05:00
thomas sarkanen
ce6660e537 Use move semantics to reduce memcpy() overhead when evaluating animation
Poses & curves are now moved instead of copied where possible.
Also removed needless passing of bone space transforms to UAnimInstance::ParallelEvaluateAnimation, which further removes an extra allocation for sub-instances.

#rb Martin.Wilson


#ROBOMERGE-SOURCE: CL 4810269 via CL 4810272 via CL 4812712

[CL 4812722 by thomas sarkanen in Main branch]
2019-01-25 11:48:55 -05:00
Lina Halper
f473c3d669 Merging //UE4/Dev-Main to Dev-Anim (//UE4/Dev-Anim)
#fyi: Laurent.Delayen, Thomas.Sarkanen
#rb: none
#lockdown thomas.sarkanen

[CL 4715236 by Lina Halper in Dev-Anim branch]
2019-01-11 21:07:09 -05:00
thomas sarkanen
349f0a4cfa Fixed anim blueprint editor viewport-linked skeletal meshes not animating
#rb Jurre.deBaare


#ROBOMERGE-SOURCE: CL 4709223

[CL 4709226 by thomas sarkanen in Main branch]
2019-01-11 10:32:28 -05:00
Thomas Sarkanen
8ba3c4c087 Merging //UE4/Dev-Main to Dev-Anim (//UE4/Dev-Anim) @ CL 4643671
#rb none
#jira none

[CL 4665410 by Thomas Sarkanen in Dev-Anim branch]
2018-12-17 06:31:16 -05:00
Ben Marsh
7598af0532 Update copyright notices to 2019.
#rb none
#lockdown Nick.Penwarden

[CL 4662404 by Ben Marsh in Main branch]
2018-12-14 13:41:00 -05:00
Ben Marsh
13d012685f Merging copyright update from 4.19 branch.
#rb none
#rnx
#jira

[CL 3818977 by Ben Marsh in Staging-4.19 branch]
2018-01-02 15:30:26 -05:00
Marc Audy
9d110f953e Copying //UE4/Dev-AnimPhys to //UE4/Dev-Main (Source: //UE4/Dev-AnimPhys @ 3806524) (Part2)
#lockdown Nick.Penwarden
#rb none
#rnx

============================
  MAJOR FEATURES & CHANGES
============================

Change 3783110 by Thomas.Sarkanen

	Added support for logical negation when copying to array properties in the fast path

	#jira UE-52043 - Anim fast path correctly doesn't handle copying to arrays of bools with logical negation

Change 3783112 by Thomas.Sarkanen

	Fixed not being able to assign/unassign constraint and physical animation profiles from the context menu

	#jira UE-50205 - Constraint cannot be assigned or unassigned from current profile when using "Unassign" from the context menu in the Skeleton Tree and Graph

Change 3783114 by Thomas.Sarkanen

	Asset picker now only reports 'picks' on user interactions

	In Ocean and other projects with lots of animation assets, the asset picker amortizes its filter queries over a number of frames. This causes the list to temporarily not include the currently selected item, which then gets reported to client code as a 'deselection'. To address this (and to maintain backwards compatibility) I've added a new delegate that gives more context for selections to the SAssetPicker, allowing us to not report selections that are 'Direct'.

	#jira UE-46802 - Searching for an animation from the dropdown in persona causes it to instantly close

Change 3783118 by Thomas.Sarkanen

	Added inline time/frame settings to notify context menu

	#jira UE-45410 - Inline MontageSection/Notify Time Entry

Change 3783122 by Thomas.Sarkanen

	Collision Response dropdown is now a checkbox

	Added details customization to allow the enum to masquerate as a bool property

	#jira UE-47916 - Drop down used for Collision Response enable/disable in Physics Asset Editor

Change 3783183 by Jurre.deBaare

	Follow up fix for toggling post processing in the preview scene

Change 3783186 by Jurre.deBaare

	Material Baking Options has two Mesh Settings sections
	#fix Changed category names for to allow for two distinct categories (would have preferred to merge them, but am limited by the way detailsview handles multiple objects with containing same category)
	#jira UE-52645

Change 3783188 by Jurre.deBaare

	Duplicating:
	"Simplygon Skelmesh "Regenerate" button remains for content that had Lods created in simplygon, even when it's not installed. On regenerating editor crashes - FPersonaMeshDetails::ApplyChanges()
	#fix Ensure the regenerate button is disabled when there is no MeshReduction interface available
	#jira UE-52641

Change 3783205 by Jurre.deBaare

	Bounds are not shown correctly in Persona for imported alembic skeletal mesh
	#fix Ensured that we do not add invalid of (0,0,0) bounds to outgoing mesh bounds
	#jira UE-49338

Change 3783248 by Jurre.deBaare

	Preview Scene Settings window is missing Search bar
	#fix changed search bar flag on Details View
	#jira UE-50063

Change 3783267 by Jurre.deBaare

	Animation Modifiers: GetBonePosesForTime does not create valid data for GetBonePose
	#fix changed the way we retrieve bone transforms, now take it directly from the raw data rather than from a pose
	#jira UE-52057

Change 3783281 by Jurre.deBaare

	Tool Tip issues in Animation Editor Preferences
	#fix corrected typos in comments
	#jira UE-51338

Change 3783373 by Thomas.Sarkanen

	Added error-reporting to the profile name widgets

	This means that the effect of renaming profiles the same as an existing one is clear (i.e. we deny it now).

	#jira UE-48120 - Renaming Physics profiles the same as existing Physics profiles switches profiles and is confusing

Change 3783438 by Jurre.deBaare

	Vertex paint fill tool fills all channels
	#fix Ensure that the current channel fill selection in the Mesh Painter is also used for the fill tool, to allow the user masking out certain channels when using fill
	#jira UE-49256

Change 3783583 by Thomas.Sarkanen

	Correctly return whether a mesh section is shown or not when it has not been edited

	This fixes skeletal->static mesh conversion

Change 3783598 by Thomas.Sarkanen

	Fix multi-convex generastion for skeletal meshes

	Some triangle indices were being missed out of the index buffer, so the mesh that the convex hull was generated from was corrupt. Removing an early-out fixes this.

	#jira UE-52529 - Inaccurate Collision is Generated When Using Multi Convex Hull (PhysicsAsset Editor)

Change 3783615 by Jurre.deBaare

	OpacityMask/Opacity bug in MeshUtilities
	#fix ensured that material baking uses correct texture samples for Opacity Mask property
	#misc deprecated all of this functionality as users should be using the MaterialBaking module
	#jira UE-52382

Change 3783620 by Martin.Wilson

	Fix crash due to oversampling animation during compression

	#Jira UE-52713

Change 3783633 by Jurre.deBaare

	Fix deprecation warnings on CIS

Change 3783636 by Benn.Gallagher

	Fixed non-working tethers in clothing
	Fixed clothing config not applying to active simulation after editing
	Fixed and re-enabled accurate wind mode

	#jira UE-50797, UE-43611

Change 3783637 by Benn.Gallagher

	Github PR: Fix world to actor transform bug in anim dynamics

	Fixed incorrect Actor-space calculations for simulation transforms and world vector transformations inside AnimDynamics highlighted when testing the above PR

	#jira UE-48681
	#3929

Change 3783638 by Benn.Gallagher

	Fixed UBlendProfile properties not being correctly customized on anim nodes - and animation sequence references not being correctly filtered on anim nodes.

Change 3783660 by Danny.Bouimad

	Fixing #UE-40686 Mass TranslatedMass Automated test, instead of decreasing tolrence I changed the content to preserve granularity.

Change 3783974 by Ori.Cohen

	Refactor when sync components is called and how we pass the data into plugins. Simplifies how we handle physx data getting invalidated by UE4 during updates.

	Also fixed crash when scene query returns a destructible component that's been destroyed, but final apex delete flush hasn't happened yet.

	#jira UE-50215

Change 3784112 by Benn.Gallagher

	Fixed subinstance nodes inside states failing to correctly create bridge variables on their skeleton class due to not correctly processing subgraphs in the anim blueprint compiler.

	#jira UE-51000

Change 3784277 by Martin.Wilson

	Fix socket name getting an appended _0

	#jira UE-46625

Change 3785589 by Ori.Cohen

	Fix cis

Change 3786336 by Martin.Wilson

	Pushing skeleton to live link can now take source guid
	-Message bus source pushes guid when sending skeleton

Change 3786778 by Martin.Wilson

	Added ability for worker thread animation blueprint code to report to Message Log + added additive warning when playing an animation on an additive pose

	#jira UE-49780

Change 3786847 by Martin.Wilson

	Initialization and delta time for live link retargeter

	#Jira UE-52112

Change 3786852 by Lina.Halper

	Sequencer blending support

	#jira: UE-52183

Change 3786924 by Lina.Halper

	PR #4210: FIX: Incorrectly passing an unrelated bool rather than the expected pose ind. (Contributed by ruffenman)


Change 3787114 by Jurre.deBaare

	Discrepancy in description of Preview Scene setting and keyboard shortcut
	#fix Changed naming of the settings to match the Advanced Preview Scene panel
	#jira UE-50060

Change 3787115 by Jurre.deBaare

	Animation Editor Preferences do not update the preview scene
	#fix Removed unused preference from PersonaOptions
	#jira UE-51318

Change 3787117 by Jurre.deBaare

	Off-by one error in frame time calculations
	#fix Fixed up UAnimSequenceBase::GetTimeAtFrame and UAnimSequenceBase::GetFrameAtTime to return correct frame indices, similar to FAnimationRuntime::GetKeyIndicesFromTime
	#jira UE-52037

Change 3787412 by Martin.Wilson

	CIS Fix

Change 3787622 by Ethan.Geller

	Include Google Resonance SDK.

Change 3787633 by Ethan.Geller

	Promote AmibsonicsMixerPtr to FAudioDevice

Change 3788026 by Lina.Halper

	Retarget source reference to soft object ptr

	#jira: UE-48570

Change 3788252 by Ethan.Geller

	Add blueprint functions for Resonance Global Reverb

Change 3788750 by Ethan.Geller

	fix single file compile for Resonance plugin

Change 3788763 by Ethan.Geller

	include IModularFeatures.h explicitly for incremental build

Change 3789108 by Martin.Wilson

	Fix animations with scaled root bone generating incorrect root motion

	#jira UE-52088

Change 3789642 by Martin.Wilson

	Fix transition nodes from referencing the wrong state node due to non unique node guids being introduced by copy and paste

	#jira UE-43891

Change 3790165 by Martin.Wilson

	Fix marker sync position not being maintained across instant transitions

	#jira UE-21355

Change 3790182 by Ethan.Geller

	Final Resonance edits pass.

Change 3790184 by Lina.Halper

	Fix issue with crash when montage is streamed out while event is queued.

	https://udn.unrealengine.com/questions/404318/anim-montage-queued-montage-event-crash.html

Change 3790207 by dan.reynolds

	#UE-50774 Updated AEOverviewReverb to not attempt to destroy Audio Component that has already been destroyed.

Change 3790215 by Martin.Wilson

	CIS Fix

Change 3790953 by Ethan.Geller

	#jira UE-53023 bypass filters when at max frequency for LPF, DC for HPF

Change 3791832 by Martin.Wilson

	Don't load animations for preview tooltip in Persona

	#jira UE-52118

Change 3792873 by David.Hill

	Fix CIS.  Remove timer from proxylod code.

Change 3793251 by David.Hill

	ProxyLOD Thirdparty libs  build cs files.

	Update from ModuleRules.WindowsPlatform to ReadOnlyTargetRules.WindowsPlatform
	-- The WindowsPlatform alias is deprecated in 4.18

Change 3793400 by Ethan.Geller

	Update Resonance blueprint library to fit google naming conventions

Change 3794097 by Benn.Gallagher

	Fixed clothing visualizations no longer functioning
	#jira UE-52995

Change 3794250 by Danny.Bouimad

	Regenerated ground truth on LODCurveLinkingTest1 and AnimatedCloth, expected change as a result of Ben fixing a bug.
	Needed to update Owens cloth settings too.
	Should resolve automation test CIS fails

Change 3794352 by David.Hill

	ProxyLOD code:

	Disable openvdb-centric warnings within the openvdb platform.h file.
	C6326: Potential comparison of a constant with another constant

	and add annotations
	C28251: Inconsistent annotation for 'copysign'

	also added a warning suppress for static analysis CA_SUPPRESS(6011) within the proxylod version of the simplifier.

Change 3794786 by Lina.Halper

	Pose asset retarget source bug fix

	#jira: UE-52429

Change 3794841 by Danny.Bouimad

	Hopefully fixes the cloth automation CIS

Change 3795191 by Lina.Halper

	Fix build issue

Change 3795486 by Ethan.Geller

	re-enable android support for Oculus Audio

Change 3796162 by Danny.Bouimad

	Third attempt to fix the cloth CIS error. Hopefuly this will solve it.

Change 3796311 by Martin.Wilson

	Remove recompress animation from curve track actions. Allows smoother interaction on animations with slow recompress time.

	#jira UE-51740

Change 3796321 by Thomas.Sarkanen

	Duplicating CL 3770752 from 4.18:

	Prevent crash when generating convex bodies fails

	Note: speculative fix as the issue cannot be reproduced locally

	#jira UE-52449 - [CrashReport] FPhysicsAssetUtils::CreateFromSkeletalMeshInternal()

Change 3797093 by Danny.Bouimad

	Constrant node AnimBP Automated tests

Change 3797384 by Danny.Bouimad

	Fixing CIS error caused by automated test lighting issue

Change 3800621 by Thomas.Sarkanen

	Fix CIS: Shadowed variable warning

	#jira UE-53253 - FMenuBuilder declaration shadows a local variable warning appears when building the editor on Linux

Change 3800690 by Danny.Bouimad

	Checking in fix for CIS automation for ConstraintNode, set the screenshot tool to use BaseColor. This should fix the issue with the rendering fuzzyness

Change 3800874 by David.Hill

	Clean up static analysis warnings

	#jira: UE-53270

Change 3801227 by David.Hill

	Allow proxylod to fail gracefully if the input mesh is way too big (e.g. sky sphere)

	Added code to automatically compute the correct spatial sampling rate based on the geometry size, also allow the user to override.

	#cl: UE-53155

Change 3801228 by David.Hill

	UI: Mesh Proxy Dialog  re-write.  Make this more like MeshMerging, and share some code.

	#cl UE-53155, UE-52787, UE-53106

Change 3801319 by Danny.Bouimad

	Regenerated all the screen shots for the constraint tests.

Change 3801383 by Ethan.Geller

	#jira UE-53311 fix additional #if PLATFORM_WINDOWS  guards in Oculus Audio

Change 3801697 by Ethan.Geller

	include AudioDevice.h directly to resolve FAudioDevice.

Change 3802180 by David.Hill

	This should fix the Incremental UE4Editor Linux build.

Change 3802643 by David.Hill

	ProxyLOD UI change. Add limits to the target screen size.  They now reflect the values in the old version of the UI and the thirdparty tool.

	#CL: UE-53313

Change 3802986 by Ethan.Geller

	#jira UE-53330 Change vraudio to explicit library path

Change 3803448 by Danny.Bouimad

	disabling constraint tests

Change 3803678 by Danny.Bouimad

	#jira UE-53306 Fix

Change 3804333 by Ethan.Geller

	#jira UE-53330 fix library paths for iOS on Resonance

Change 3804453 by David.Hill

	Fix Shadow warning when compiling UE4Editor on linux:

	FlattenedMaterials.

	#CL: UE-53349

Change 3804510 by Lina.Halper

	CIS warning on shadow vars

	#jira: UE-53348, UE-53345

Change 3805451 by Lina.Halper

	Fix build issue : Renamed variable -

	https://ec-01.epicgames.net/commander/link/jobStepDetails/jobSteps/74095846?stepName=Incremental%20UnrealHeaderTool%20Win64&jobId=8173688&jobName=UE4%20Dev-AnimPhys%20-%20CL%203805429%20-%20Incremental%20Editor%20Win64&tabGroup=diagnosticHeader

Change 3805470 by Lina.Halper

	Fix build issue

Change 3806524 by Martin.Wilson

	Only use previous frame end position if it is valid for this frame

	#jira UE-53414

Change 3792620 by David.Hill

	Copying //UE4/Dev-ProxyLOD to Dev-AnimPhys-Minimal (//UE4/Dev-AnimPhys-Minimal)

	Adding the ProxyLOD code to AnimPhys.

Change 3796059 by Thomas.Sarkanen

	Persona viewport settings are now per-asset editor

	This prevnets bone following (etc) being shared by all Persona asset editors
	FOV & view type is no longer chared by all Persona asset editors

	#jira UE-53005 - Viewport settings like bone following are shared between all animation sub-editors

[CL 3806814 by Marc Audy in Main branch]
2017-12-13 16:02:45 -05:00