-
Notifications
You must be signed in to change notification settings - Fork 688
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
[api] Adds Safetensors support #2763
Conversation
b931f8e
to
9a02d3a
Compare
Codecov ReportPatch coverage:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## master #2763 +/- ##
============================================
+ Coverage 72.08% 72.20% +0.11%
- Complexity 5126 7086 +1960
============================================
Files 473 700 +227
Lines 21970 31534 +9564
Branches 2351 3262 +911
============================================
+ Hits 15838 22769 +6931
- Misses 4925 7212 +2287
- Partials 1207 1553 +346
☔ View full report in Codecov by Sentry. |
@@ -450,4 +542,23 @@ public String toString() { | |||
} | |||
return builder.toString(); | |||
} | |||
|
|||
/** An enum represents NDList serialization format. */ | |||
public enum Mode { |
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.
Maybe rename to Encoding
?
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.
done
Fixes #2719
Description
Brief description of what this PR is about