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

Proposal - 3.0 Additional types for Silk.NET.Maths #1806

Merged
merged 26 commits into from
Nov 30, 2024

Conversation

dfkeenan
Copy link
Contributor

Summary of the PR

A proposal for additional types to the Silk.NET.Maths library for 3.0.

Related issues, Discord discussions, or proposals

As discussed in discord with the SilkX team.

@dfkeenan
Copy link
Contributor Author

@dotnet-policy-service agree

@Perksey
Copy link
Member

Perksey commented Feb 21, 2024

Provisionally adding this to the next working group meeting agenda (date not yet set)

@Perksey Perksey added this to the Next Working Group Meeting milestone Feb 21, 2024
@uwx
Copy link

uwx commented Apr 24, 2024

Angle (and other such types) shouldn't have a public constructor, that completely voids the point of having the type as it's not clear what underlying unit is being passed in without reading the code. That's why we have FromDegrees and FromRadians. Angle.Sin(new Angle(1.0)) is no more clear than Math.Sin(1.0), but Angle.Sin(Angle.FromRadians(1.0)) is.

@Perksey
Copy link
Member

Perksey commented Nov 14, 2024

Hey! How's the progress here? :)

@dfkeenan
Copy link
Contributor Author

Hey! How's the progress here? :)

Yo!

Sorry, this got put on the back burner due to other stuff. To be honest not sure what state it is in. 😅

I do remember wanting to discuss some things about the design with the team. Looking at the proposals the Angle and Color proposals look pretty close, There does appear to be some obvious blank spots in the Collision proposal.

Might be worth getting some feed back from some other people. Not sure if you want to bring it to the next 3.0 meeting.

@dfkeenan
Copy link
Contributor Author

The proposals I started aren't quite there yet but I don't want to keep working on them at the moment. I have marked them as work in progress so perhaps someone else can take over.

@dfkeenan dfkeenan marked this pull request as ready for review November 30, 2024 11:17
@dfkeenan dfkeenan requested a review from a team as a code owner November 30, 2024 11:17
@Perksey Perksey enabled auto-merge (squash) November 30, 2024 11:18
@Perksey Perksey disabled auto-merge November 30, 2024 11:18
@Perksey Perksey merged commit 8da3988 into dotnet:main Nov 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants