Skip to content

Commit

Permalink
format code
Browse files Browse the repository at this point in the history
  • Loading branch information
snnn committed Feb 13, 2025
1 parent 312b404 commit 6920f53
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 8 deletions.
2 changes: 0 additions & 2 deletions onnxruntime/test/ir/onnx_model_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ class ONNXModelsTest : public ::testing::Test {
std::unique_ptr<logging::Logger> logger_;
};


TEST_F(ONNXModelsTest, non_existing_model) {
// NOTE: this requires the current directory to be where onnxruntime_ir_UT.exe is located
std::shared_ptr<Model> model;
Expand Down Expand Up @@ -60,7 +59,6 @@ class ONNXModelsTest1 : public ::testing::TestWithParam<const ORTCHAR_T*> {
}
};


// test a model that conforms to ONNX IR v4 where there are initializers that are not graph inputs.
// a NodeArg should be created for all initializers in this case.
// the test model contains initializers that are used as implicit inputs in a subgraph, and the NodeArg is required
Expand Down
3 changes: 1 addition & 2 deletions onnxruntime/test/onnx/TestCase.cc
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,6 @@ void OnnxTestCase::LoadTestData(size_t id, onnxruntime::test::HeapBuffer& b,

std::vector<std::filesystem::path> test_data_pb_files = SimpleGlob(test_data_dirs_[id], is_input ? ORT_TSTR("input_*.pb") : ORT_TSTR("output_*.pb"));


SortFileNames(test_data_pb_files);

for (size_t i = 0; i < test_data_pb_files.size(); ++i) {
Expand Down Expand Up @@ -594,7 +593,7 @@ void LoadTests(const std::vector<std::basic_string<PATH_CHAR_TYPE>>& input_paths
}

#if !defined(ORT_MINIMAL_BUILD)
// The for-loop below needs to load every ONNX model into memory then destory the in-memory objects, which is very inefficient since 1. in total we need to load every model twice 2. at here we do the job sequentially.
// The for-loop below needs to load every ONNX model into memory then destory the in-memory objects, which is very inefficient since 1. in total we need to load every model twice 2. at here we do the job sequentially.

Check warning on line 596 in onnxruntime/test/onnx/TestCase.cc

View workflow job for this annotation

GitHub Actions / Optional Lint

[misspell] reported by reviewdog 🐶 "destory" is a misspelling of "destroy" Raw Output: ./onnxruntime/test/onnx/TestCase.cc:596:72: "destory" is a misspelling of "destroy"
// Originally the design was to make the TestModelInfo lightweight so that all the model information can be retrieved from filesystem meta data without actually loading the models.
for (const std::filesystem::path& model_path : onnx_models) {
LoadSingleModel(TestModelInfo::LoadOnnxModel(model_path), tolerances, broken_tests, broken_tests_keyword_set, process_function);
Expand Down
4 changes: 0 additions & 4 deletions onnxruntime/test/onnx/TestCase.h
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,6 @@ namespace Ort {
struct Value;
}



namespace onnxruntime {
namespace test {
class HeapBuffer;
Expand Down Expand Up @@ -48,8 +46,6 @@ class ITestCase {
virtual void GetPostProcessing(bool* value) const = 0;
};



std::unique_ptr<ITestCase> CreateOnnxTestCase(const std::string& test_case_name,
std::unique_ptr<TestModelInfo> model,
double default_per_sample_tolerance,
Expand Down

0 comments on commit 6920f53

Please sign in to comment.