Skip to content

Finalize Maths API #215

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

Closed
wants to merge 10 commits into from
Closed

Finalize Maths API #215

wants to merge 10 commits into from

Conversation

HurricanKai
Copy link
Member

@HurricanKai HurricanKai commented Jul 1, 2020

Currently just a draft to track progress of what's done here, see #190 for what was done in other PRs and what will be done in other PRs.

  • Code Cleanup (currently ~50%)
  • Matrix4X4
  • Matrix4X3
  • Matrix4X2
  • Matrix3X4
  • Matrix3X3
  • Matrix3X2
  • Matrix2X3
  • Matrix2X2
  • Code generator for void Func(ref T1 a, T2 b, out TResult result
  • Helper Program to convert a * b - c / d + e style expressions to Scalar based expressions (may be removed in the future)

Note that a bunch of stuff is broken, and the automatic code cleanup part even ended up removing some (all?) license notices, added regions where they are not supposed to be etc. etc. etc. so this won't reflect what the types will look like in the end.

@Perksey
Copy link
Member

Perksey commented Jul 2, 2020

Yeah I'd imagine changing up the style for the maths project has really fucked some of the actually meaningful settings XD like the license header and #regions bad to name a couple

@HurricanKai
Copy link
Member Author

yeah, I didn't actually think those settings would be affected, but I believe some of those bad changes also were dotnet-format misinterpreting settings

@HurricanKai
Copy link
Member Author

PublicAPI/**/PublicAPI.(Un)shipped.txt is now tracking API surface

@HurricanKai
Copy link
Member Author

Will be replaced

@HurricanKai HurricanKai deleted the feature/matrices branch July 13, 2020 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants