Skip to content
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

Setup a basic SystemSet enum for the game #10

Closed
janhohenheim opened this issue Jul 4, 2024 · 2 comments · Fixed by #70
Closed

Setup a basic SystemSet enum for the game #10

janhohenheim opened this issue Jul 4, 2024 · 2 comments · Fixed by #70
Labels
enhancement New feature or request
Milestone

Comments

@janhohenheim
Copy link
Member

Migrated from bevyengine/bevy_github_ci_template#63

Same rationale as #4

@janhohenheim janhohenheim changed the title Setup a basic SystemSet enum for the game #63 Setup a basic SystemSet enum for the game #63 Jul 4, 2024
@janhohenheim janhohenheim changed the title Setup a basic SystemSet enum for the game #63 Setup a basic SystemSet enum for the game Jul 4, 2024
@benfrankel
Copy link
Collaborator

This is big bikeshed territory. I can share my own game logic system set enum:

/// (Update) Game logic system ordering
#[derive(SystemSet, Clone, Eq, PartialEq, Hash, Debug)]
pub enum UpdateSet {
    /// Handle actions pressed this frame
    HandleActions,
    /// Apply deferred effects from HandleActions
    HandleActionsFlush,
    /// Initialize start-of-frame values and tick timers
    Start,
    /// Step game logic
    Update,
    /// Run the trigger-effect system
    React,
    /// Record player and AI intents
    RecordIntents,
    /// Apply player and AI intents
    ApplyIntents,
    /// Handle events emitted this frame
    HandleEvents,
    /// Queue despawn commands from DespawnSet
    QueueDespawn,
    /// Apply spawn / despawn and other commands
    ApplyDeferred,
    /// Update UI
    UpdateUi,
    /// Synchronize end-of-frame values
    End,
}

This could definitely be stripped down. I'm curious if there are commonalities between the system set enums in different template repos.

@benfrankel benfrankel added the enhancement New feature or request label Jul 4, 2024
@MiniaczQ
Copy link
Collaborator

MiniaczQ commented Jul 5, 2024

It's hard to setup any ordering without providing game logic, closely bound to #11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants