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

Make osu!framework use the ECS Paradigm #210

Closed
sr229 opened this issue Jun 8, 2021 · 4 comments
Closed

Make osu!framework use the ECS Paradigm #210

sr229 opened this issue Jun 8, 2021 · 4 comments
Assignees
Labels
priority:high Bugs or Features that requires immediate attention.

Comments

@sr229
Copy link
Member

sr229 commented Jun 8, 2021

As it stands right now there's some things fundamentally broken which is a major roadblock on Vignette. We're going to fix one of them, and one of those is the draw hierarchy, because right now the inheritance in the hierarchy doesn't make sense.

@sr229 sr229 added priority:high Bugs or Features that requires immediate attention. area:osu-framework labels Jun 8, 2021
@sr229 sr229 added this to To do in Debut 2 via automation Jun 8, 2021
@sr229 sr229 moved this from To do to In progress in Debut 2 Jun 8, 2021
@sr229
Copy link
Member Author

sr229 commented Jun 8, 2021

TODO

  • Implement Component and Composite<T>
  • Implement osu.Framework.Primitives.
  • Port everything that belongs as a Primitive on said namespace

More guidance at ppy/osu-framework#4281

@sr229 sr229 mentioned this issue Jun 14, 2021
2 tasks
@sr229 sr229 removed this from In progress in Debut 2 Sep 9, 2021
@sr229 sr229 added this to To do in Backlog via automation Sep 9, 2021
@sr229 sr229 removed this from To do in Backlog Sep 9, 2021
@sr229 sr229 added this to To do in Encore via automation Sep 9, 2021
@sr229
Copy link
Member Author

sr229 commented Sep 9, 2021

Assigned to Encore 1

@LeNitrous LeNitrous changed the title Fix draw hierarchy Make osu!framework use the ECS Paradigm Sep 9, 2021
@LeNitrous
Copy link
Member

Changed the title as it affects more than the draw hierarchy at this point, after a discussion with @sr229 here's the proposed structure:
image

@sr229
Copy link
Member Author

sr229 commented Sep 13, 2021

We have committed on Encore to use a different approach. Closing as wontfix.

@sr229 sr229 closed this as completed Sep 13, 2021
Encore automation moved this from To do to Done Sep 13, 2021
@sr229 sr229 removed this from Done in Encore Sep 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:high Bugs or Features that requires immediate attention.
Projects
None yet
Development

No branches or pull requests

2 participants