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
Fix FaceFX Sequence TearDown on LevelSequencePlayer::PlayToFrame #171
base: master
Are you sure you want to change the base?
Conversation
Two FaceFX in sequence is playing and LevelSequencePlayer call PlayToFrame. This occurs sequence template TearDown called both of. First TearDown call TrackData.Empty() (TrackData is TMap container) at the END. Second TearDown will no idea to do anything which TrackData is already EMPTY. This commit is fix to remove from TrackData depend on current FaceFX Track Guid.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey Cloud,
thanks for the PR. Do you have a repro case where we can see the issue with ? What are the steps to run into this issue ?
It's my company's developing project which means I couldn't give the repo. Maybe I can take some screenshots of this issue for you tomorrow. |
@@ -155,6 +156,7 @@ void FFaceFXAnimationSectionTemplate::TearDown(FPersistentEvaluationData& Persis | |||
return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found another problem while editing.
TrackId will not remove from TarckData on sequence edit due to function returned by key is nullptr.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It caused by lost reference of Actor.
I opened level which actor in and play again, the Key is not nullptr
Is it possible to set up a sample scene using our sample content that will reproduce these issues? |
I will trying to reproduce this issue on simple project. |
Two FaceFX in sequence is playing and LevelSequencePlayer call PlayToFrame.
This occurs sequence template TearDown called both of.
First TearDown call TrackData.Empty() (TrackData is TMap container) at the END.
Second TearDown will no idea to do anything which TrackData is already EMPTY.
This commit is fix to remove from TrackData depend on current FaceFX Track Guid.