mirror of
https://github.com/pytorch/pytorch.git
synced 2025-10-21 05:34:18 +08:00
3d37f5b052
Summary: Pull Request resolved: https://github.com/pytorch/pytorch/pull/72596 debug_pkl file inside of pytorch's .pt file consists of a list of SourceRanges. Each SourceRange points to a Source which is a stack track, filename, and start, end numbers. Those are emitted in debug_pkl file as strings. Since many SourceRange shares the same source, the string for trace can be deduped. The newer format saves a set of unique traces in a tuple, then each SourceRange will save the offset of it's trace w.r.t. position in that tuple. (i.e. manually applying dictionary compression). The above helps with smaller file size. On loading, if we copy each trace to Source as string the runtime memory would still blowup. To mitigate this, we use SourceView directly instead of source which will take the reference of string inside of Deserializer and make that into string_view. This is safe because Deserializer is hold by Unpickler by shared_ptr, and Unpickler is also hold by shared_ptr by another Source object. That Source object will be alive during the model construction. Test Plan: unit test Took original file (312271638_930.predictor.disagg.local); loaded with `torch.jit.load` save again with `torch.jit.save`. Unzip both, look at contents: ``` [qihan@devvm5585.vll0 ~]$ du archive -h 4.0K archive/xl_model_weights 3.7M archive/extra 8.0K archive/code/__torch__/caffe2/torch/fb/model_transform/splitting 8.0K archive/code/__torch__/caffe2/torch/fb/model_transform 8.0K archive/code/__torch__/caffe2/torch/fb 8.0K archive/code/__torch__/caffe2/torch 8.0K archive/code/__torch__/caffe2 20M archive/code/__torch__/torch/fx/graph_module 20M archive/code/__torch__/torch/fx 8.0K archive/code/__torch__/torch/classes 20M archive/code/__torch__/torch 20M archive/code/__torch__ 20M archive/code 2.7M archive/constants 35M archive [qihan@devvm5585.vll0 ~]$ du resaved -h 4.0K resaved/extra 8.0K resaved/code/__torch__/caffe2/torch/fb/model_transform/splitting 8.0K resaved/code/__torch__/caffe2/torch/fb/model_transform 8.0K resaved/code/__torch__/caffe2/torch/fb 8.0K resaved/code/__torch__/caffe2/torch 8.0K resaved/code/__torch__/caffe2 1.3M resaved/code/__torch__/torch/fx/graph_module 1.3M resaved/code/__torch__/torch/fx 8.0K resaved/code/__torch__/torch/classes 1.4M resaved/code/__torch__/torch 1.4M resaved/code/__torch__ 1.4M resaved/code 2.7M resaved/constants 13M resaved [qihan@devvm5585.vll0 ~]$ ``` Reviewed By: JasonHanwen Differential Revision: D33994011 fbshipit-source-id: 8e6224c6e942e91c3403f686c8f0937d1002ed41 (cherry picked from commit a7014dd4029308c95007f362a57c31796d686647)
33 lines
987 B
C++
33 lines
987 B
C++
#include <gtest/gtest.h>
|
|
#include <torch/csrc/jit/frontend/source_range.h>
|
|
|
|
using namespace ::testing;
|
|
using namespace ::torch::jit;
|
|
|
|
TEST(SourceRangeTest, test_find) {
|
|
std::vector<std::shared_ptr<std::string>> strings;
|
|
strings.push_back(std::make_shared<std::string>("hello world"));
|
|
strings.push_back(std::make_shared<std::string>("nihaoma"));
|
|
|
|
std::vector<c10::string_view> pieces{*strings[0], *strings[1]};
|
|
|
|
StringCordView view(pieces, strings);
|
|
|
|
auto x = view.find("rldni", 0);
|
|
EXPECT_EQ(x, 8);
|
|
}
|
|
|
|
TEST(SourceRangeTest, test_substr) {
|
|
std::vector<std::shared_ptr<std::string>> strings;
|
|
strings.push_back(std::make_shared<std::string>("hello world"));
|
|
strings.push_back(std::make_shared<std::string>("nihaoma"));
|
|
|
|
std::vector<c10::string_view> pieces{*strings[0], *strings[1]};
|
|
|
|
StringCordView view(pieces, strings);
|
|
|
|
auto x = view.substr(4, 10).str();
|
|
EXPECT_EQ(x, view.str().substr(4, 10));
|
|
EXPECT_EQ(view.substr(0, view.size()).str(), view.str());
|
|
}
|