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

Major breaking changes tracking #2576

Open
1 of 10 tasks
Doprez opened this issue Dec 22, 2024 · 2 comments
Open
1 of 10 tasks

Major breaking changes tracking #2576

Doprez opened this issue Dec 22, 2024 · 2 comments
Labels
enhancement New feature or request
Milestone

Comments

@Doprez
Copy link
Contributor

Doprez commented Dec 22, 2024

Purpose

This issue will be an overview of planned changes that will be breaking to Strides core. This will allow us to consolidate discussions, PRs and issues into a single place for future planning.

Current PRs

Current Discussions

Tasks

TBD I will fill this out better with breakdowns for what is needed to get going on the above discussions

Math

  • Deprecate Stride math extensions in favour of System.Numerics
  • Change core math types to the System.Numerics equivelent
  • Add Implicit conversions to System.Numerics

Game startup

  • Change to a more robust GameWindow
  • decouple the GameWindow from GamePlatform maybe in favour of something that defines just the update/render loop?
  • remove GameContext it serves no real purpose and just adds an unnecessary layer. Should use GameSettings instead
  • remove cyclical dependencies between systems, discussed here
  • Move Game.IsMouseVisible out of the GameWindow and into the Input library.

Graphics

branch chores

  • Make a new dedicated branch for proper testing (5.0?)
@Doprez Doprez added the enhancement New feature or request label Dec 22, 2024
@Jklawreszuk
Copy link
Collaborator

You can also add my two PRs which can be considered as breaking change:

@Doprez
Copy link
Contributor Author

Doprez commented Jan 9, 2025

I added the directx 9 and opengl 2 support to the main checklist but I think @xen2 wasn't convinced to remove metrics last I remember?

I also dont think its a breaking change so Im not sure that we would need to wait for 5.0 if we wanted to merge that in.

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

No branches or pull requests

3 participants