Skip to content

[torch api] Enable _TORCH_ONNX_SAVE_EXTERNAL_DATA_WITH_IR#1866

Merged
justinchuby merged 1 commit intomainfrom
justinchu/external-data-flag
Sep 13, 2024
Merged

[torch api] Enable _TORCH_ONNX_SAVE_EXTERNAL_DATA_WITH_IR#1866
justinchuby merged 1 commit intomainfrom
justinchu/external-data-flag

Conversation

@justinchuby
Copy link
Copy Markdown
Collaborator

Enable the _TORCH_ONNX_SAVE_EXTERNAL_DATA_WITH_IR flag to use the new external data logic. This will

  • Reduce peak memory usage
  • Align external data to 64k

for the torch exporter.

Enable the `_TORCH_ONNX_SAVE_EXTERNAL_DATA_WITH_IR` flag to use the new external data logic. This will

- Reduce peak memory usage
- Align external data to 64k

for the torch exporter.
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.42%. Comparing base (a99e443) to head (95800f8).

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1866      +/-   ##
==========================================
+ Coverage   75.26%   75.42%   +0.16%     
==========================================
  Files         251      251              
  Lines       27446    27446              
  Branches     5032     5032              
==========================================
+ Hits        20656    20702      +46     
+ Misses       5822     5778      -44     
+ Partials      968      966       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@justinchuby justinchuby merged commit 377869a into main Sep 13, 2024
@justinchuby justinchuby deleted the justinchu/external-data-flag branch September 13, 2024 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants