Skip to content

AwakePriority virtual property#1093

Open
microlith57 wants to merge 2 commits intoEverestAPI:devfrom
microlith57:awake-priority
Open

AwakePriority virtual property#1093
microlith57 wants to merge 2 commits intoEverestAPI:devfrom
microlith57:awake-priority

Conversation

@microlith57
Copy link
Member

@microlith57 microlith57 commented Mar 15, 2026

this adds an AwakePriority virtual property on Entity, that can be used to change the order that Awake is called, similar to how Depth affects update and render.

  • thanks to bit (micycle on codeberg) for the initial implementation!
  • thanks to @cloudsbelow for suggesting the virtual property approach (chosen since it doesn't cost memory or performance per entity, while a component or field would)

testmod build: microlith57/CelesteTestMod#2 (check inside the workflow run)

@maddie480-bot maddie480-bot added the 1: review needed This PR needs 2 approvals to be merged (bot-managed) label Mar 15, 2026
@SilverDorian46
Copy link
Contributor

while I don't believe in a feature necessarily having to supersede a different one, I'm just wondering...
what are your thoughts on the PostAwake event as implemented in #1055?
I'd also appreciate it if you'd review that PR as well - it's been left in the dark for quite a bit

@microlith57 microlith57 marked this pull request as draft March 16, 2026 04:27
@maddie480-bot maddie480-bot added 0: draft This PR is not ready for review yet (bot-managed) and removed 1: review needed This PR needs 2 approvals to be merged (bot-managed) labels Mar 16, 2026
@microlith57 microlith57 changed the title AwakePriority component AwakePriority virtual property Mar 17, 2026
@microlith57
Copy link
Member Author

@SilverDorian46 i agree that this serves a very similar purpose to PostAwake, but i think they're different enough to peacefully coexist ^_^

@microlith57 microlith57 marked this pull request as ready for review March 17, 2026 05:11
@maddie480-bot maddie480-bot added 1: review needed This PR needs 2 approvals to be merged (bot-managed) and removed 0: draft This PR is not ready for review yet (bot-managed) labels Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1: review needed This PR needs 2 approvals to be merged (bot-managed)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants