-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Remove the experimental attribute from the tensor APIs #117103
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
base: main
Are you sure you want to change the base?
Conversation
Should the table at the end of this document be updated to note that the experimental attribute was removed in .NET 10: https://github.com/dotnet/runtime/blob/main/docs/project/list-of-diagnostics.md If so, you might want to also update SYSLIB5005 since experimental was removed from that API in #113021. |
1e75260
to
5e0684d
Compare
7733b83
to
253bd41
Compare
cd11990
to
1cbd243
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR removes the experimental attributes from the tensor APIs and associated types, indicating that these APIs have now reached stability. Key changes include:
- Removal of [Experimental(…)] attributes from files such as Tensor.cs, TensorSpan.cs, TensorSpanDebugView.cs, ITensor*.cs, and related files.
- Updates to constructors and method signatures to consistently pass the new pinned parameter where needed.
- Adjustments to compatibility suppressions and project files to reflect these API changes.
Reviewed Changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
src/tools/illink/src/linker/CompatibilitySuppressions.xml | Updated suppression entries to align with the removal of experimental attributes. |
src/libraries/System.Numerics.Tensors/src/System/Numerics/Tensors/netcore/Tensor_1.cs | Removed the experimental attribute from Tensor and adjusted constructor signatures. |
src/libraries/System.Numerics.Tensors/src/System/Numerics/Tensors/netcore/TensorSpan_1.cs | Removed the experimental attribute and updated constructors to include the pinned parameter. |
src/libraries/System.Numerics.Tensors/src/System/Numerics/Tensors/netcore/TensorSpanDebugView.cs | Removed the experimental attribute. |
Other files | Similar removals of the experimental attribute and corresponding adjustments across different tensor APIs and supporting types. |
No description provided.