Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

removed Engine.instance.priorityAvatarEntities #8824

Merged
merged 3 commits into from
Sep 19, 2023

Conversation

anselm
Copy link
Collaborator

@anselm anselm commented Sep 17, 2023

Summary

A summary of changes being made in this PR

References

closes #insert number here

Checklist

  • If this PR is still a WIP, convert to a draft
  • When this PR is ready, mark it as "Ready for review"
  • ensure all checks pass
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewer

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

🤖 Generated by Copilot at b686b15

Summary

🗑️🔄🧹

The pull request refactors the AvatarAnimationSystem class to use the priorityQueue object instead of the Engine.instance.priorityAvatarEntities property for managing the animation of priority avatars. It also removes the unused and redundant property from the Engine class and cleans up some code.

No more priority entities
We have a better way
We use the priority queue
To animate the day

Walkthrough

  • Remove Engine.instance.priorityAvatarEntities property and use priorityQueue object instead (link, link, link, link, link)

@HexaField HexaField merged commit 39d5bc8 into dev Sep 19, 2023
7 of 8 checks passed
@HexaField HexaField deleted the deglobalize-priority-avatar-entities branch September 19, 2023 02:33
barankyle pushed a commit that referenced this pull request Oct 13, 2023
* removed Engine.instance.priorityAvatarEntities

* speedup by using for loop rather than foreach
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants