Skip to content

Enhance rstsr-common API and add serialization for flags#76

Merged
ajz34 merged 4 commits intoRESTGroup:masterfrom
ajz34:flags-serde-result-change
Apr 10, 2026
Merged

Enhance rstsr-common API and add serialization for flags#76
ajz34 merged 4 commits intoRESTGroup:masterfrom
ajz34:flags-serde-result-change

Conversation

@ajz34
Copy link
Copy Markdown
Member

@ajz34 ajz34 commented Apr 10, 2026

API breaking changes

  • Refactored the flip methods for FlagSide and FlagUpLo enums to return the flipped value directly instead of a Result, since the operation cannot fail. Updated all call sites to remove unnecessary ? error propagation.

Enhancements

  • Re-exported the BLAS traits prelude under the new feature in rstsr/src/prelude.rs for easier access.
  • Made all BLAS flag enums (FlagOrder, TensorIterOrder, FlagTrans, FlagSide, FlagUpLo, FlagDiag, FlagSymm) derive Serialize and Deserialize, and added Serde rename attributes for better control over their string representations.

Crate structure

  • Added serde as a dependency in the relevant Cargo.toml files and imported the necessary traits in flags.rs.
  • Symlinked CHANGELOG.md to rstsr crate directory.

(Partially summarized by Copilot)

@ajz34 ajz34 merged commit 81fc854 into RESTGroup:master Apr 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant