Skip to content

[caffe2] update load_save_test.py to also verify the chunking behavior#53401

Closed
simpkins wants to merge 2 commits intogh/simpkins/4/basefrom
gh/simpkins/4/head
Closed

[caffe2] update load_save_test.py to also verify the chunking behavior#53401
simpkins wants to merge 2 commits intogh/simpkins/4/basefrom
gh/simpkins/4/head

Conversation

@simpkins
Copy link
Copy Markdown
Contributor

@simpkins simpkins commented Mar 5, 2021

Stack from ghstack:

This is a reland of D26641599 after rebasing onto D26802576.

Add some small utility functions to read the blob names back from the minidb
file so that we can verify how many chunks were written for each blob.

Differential Revision: D26853942

This is a reland of D26641599 after rebasing onto D26802576.

Add some small utility functions to read the blob names back from the minidb
file so that we can verify how many chunks were written for each blob.

Differential Revision: [D26853942](https://our.internmc.facebook.com/intern/diff/D26853942/)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Mar 5, 2021

💊 CI failures summary and remediations

As of commit 3815beb (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

…ing behavior"

This is a reland of D26641599 after rebasing onto D26802576.

Add some small utility functions to read the blob names back from the minidb
file so that we can verify how many chunks were written for each blob.

Differential Revision: [D26853942](https://our.internmc.facebook.com/intern/diff/D26853942/)

[ghstack-poisoned]
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 10, 2021

Codecov Report

Merging #53401 (3815beb) into gh/simpkins/4/base (2ae6940) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@                  Coverage Diff                   @@
##           gh/simpkins/4/base   #53401      +/-   ##
======================================================
- Coverage               77.63%   77.63%   -0.01%     
======================================================
  Files                    1869     1869              
  Lines                  182377   182377              
======================================================
- Hits                   141594   141593       -1     
- Misses                  40783    40784       +1     

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 023948e.

@facebook-github-bot facebook-github-bot deleted the gh/simpkins/4/head branch March 14, 2021 14:14
xsacha pushed a commit to xsacha/pytorch that referenced this pull request Mar 31, 2021
pytorch#53401)

Summary:
Pull Request resolved: pytorch#53401

This is a reland of D26641599 (pytorch@cd9ac54) after rebasing onto D26802576 (pytorch@f595ba1).

Add some small utility functions to read the blob names back from the minidb
file so that we can verify how many chunks were written for each blob.
ghstack-source-id: 123567033

Test Plan: buck test caffe2/caffe2/python/operator_test:load_save_test

Reviewed By: mraway

Differential Revision: D26853942

fbshipit-source-id: 0b45078fdd279f547752c8fdb771e296374a00da
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
pytorch#53401)

Summary:
Pull Request resolved: pytorch#53401

This is a reland of D26641599 (pytorch@6f589c9) after rebasing onto D26802576 (pytorch@2fad0d9).

Add some small utility functions to read the blob names back from the minidb
file so that we can verify how many chunks were written for each blob.
ghstack-source-id: 123567033

Test Plan: buck test caffe2/caffe2/python/operator_test:load_save_test

Reviewed By: mraway

Differential Revision: D26853942

fbshipit-source-id: 0b45078fdd279f547752c8fdb771e296374a00da
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