Skip to content

Conversation

MatthewSandfordImprobable
Copy link
Contributor

@MatthewSandfordImprobable MatthewSandfordImprobable commented Nov 26, 2019

Issue:
There are two paths which ask to spawn a default character actor in BP_GDK_PlayerController.
When many SimPlayers clients are run within a short window, for each client, both of these paths are hit.

Fix:
I have made it so that once one path has been hit, we can no longer attempt to spawn another default actor.

Testing:
Tested running 10 SimPlayers locally. Before the fix 20 actors were created. After the fix the correct number (10). I have a few batch files that will aid in testing this. They will be added in another PR.

@m-samiec

@improbable-prow-robot
Copy link

Corresponding JIRA ticket: https://improbableio.atlassian.net/browse/UNR-2425

@improbable-prow-robot improbable-prow-robot added jira/UNR size/XS Denotes a PR that changes 0-14 lines, ignoring generated files. labels Nov 26, 2019
Copy link
Contributor

@oblm oblm left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for testing

@MatthewSandfordImprobable MatthewSandfordImprobable merged commit cc2b1fe into 0.8.0-preview-rc Nov 28, 2019
@m-samiec m-samiec deleted the bugfix/UNR-2425 branch December 9, 2019 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/UNR size/XS Denotes a PR that changes 0-14 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants