Skip to content

Commit b73796b

Browse files
committed
chore: linter fixes
Signed-off-by: Dheeraj Peri <[email protected]>
1 parent 39fc2dd commit b73796b

File tree

7 files changed

+22
-19
lines changed

7 files changed

+22
-19
lines changed

BUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@ pkg_tar(
2222
"//core/lowering:include",
2323
"//core/lowering/passes:include",
2424
"//core/partitioning:include",
25-
"//core/partitioning/segmentedblock:include",
26-
"//core/partitioning/partitioninginfo:include",
2725
"//core/partitioning/partitioningctx:include",
26+
"//core/partitioning/partitioninginfo:include",
27+
"//core/partitioning/segmentedblock:include",
2828
"//core/plugins:impl_include",
2929
"//core/plugins:include",
3030
"//core/runtime:include",

tests/cpp/test_collections.cpp

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -188,8 +188,10 @@ TEST(CppAPITests, TestCollectionTupleInputOutput) {
188188
auto trt_mod = torch_tensorrt::torchscript::compile(mod, compile_settings);
189189
auto trt_out = trt_mod.forward(complex_inputs);
190190

191-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(out.toTuple()->elements()[0].toTensor(), trt_out.toTuple()->elements()[0].toTensor()));
192-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(out.toTuple()->elements()[1].toTensor(), trt_out.toTuple()->elements()[1].toTensor()));
191+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
192+
out.toTuple()->elements()[0].toTensor(), trt_out.toTuple()->elements()[0].toTensor()));
193+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
194+
out.toTuple()->elements()[1].toTensor(), trt_out.toTuple()->elements()[1].toTensor()));
193195
}
194196

195197
TEST(CppAPITests, TestCollectionListInputOutput) {
@@ -247,8 +249,10 @@ TEST(CppAPITests, TestCollectionListInputOutput) {
247249
auto trt_mod = torch_tensorrt::torchscript::compile(mod, compile_settings);
248250
auto trt_out = trt_mod.forward(complex_inputs);
249251

250-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(out.toList().vec()[0].toTensor(), trt_out.toList().vec()[0].toTensor()));
251-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(out.toList().vec()[1].toTensor(), trt_out.toList().vec()[1].toTensor()));
252+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
253+
out.toList().vec()[0].toTensor(), trt_out.toList().vec()[0].toTensor()));
254+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
255+
out.toList().vec()[1].toTensor(), trt_out.toList().vec()[1].toTensor()));
252256
}
253257

254258
TEST(CppAPITests, TestCollectionComplexModel) {
@@ -306,6 +310,8 @@ TEST(CppAPITests, TestCollectionComplexModel) {
306310
auto trt_mod = torch_tensorrt::torchscript::compile(mod, compile_settings);
307311
auto trt_out = trt_mod.forward(complex_inputs);
308312

309-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(out.toTuple()->elements()[0].toTensor(), trt_out.toTuple()->elements()[0].toTensor()));
310-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(out.toTuple()->elements()[1].toTensor(), trt_out.toTuple()->elements()[1].toTensor()));
313+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
314+
out.toTuple()->elements()[0].toTensor(), trt_out.toTuple()->elements()[0].toTensor()));
315+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
316+
out.toTuple()->elements()[1].toTensor(), trt_out.toTuple()->elements()[1].toTensor()));
311317
}

tests/cpp/test_compiled_modules.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ TEST_P(CppAPITests, CompiledModuleIsClose) {
4141
}
4242

4343
for (size_t i = 0; i < trt_results.size(); i++) {
44-
ASSERT_TRUE(
45-
torch_tensorrt::tests::util::cosineSimEqual(jit_results[i], trt_results[i].reshape_as(jit_results[i])));
44+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(jit_results[i], trt_results[i].reshape_as(jit_results[i])));
4645
}
4746
}
4847

tests/cpp/test_default_input_types.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,5 +116,4 @@ TEST_P(CppAPITests, InputsRespectUserSettingFP32WeightsFP16In) {
116116
INSTANTIATE_TEST_SUITE_P(
117117
CompiledModuleForwardIsCloseSuite,
118118
CppAPITests,
119-
testing::Values(
120-
PathAndInput({"tests/modules/resnet18_traced.jit.pt", {{1, 3, 224, 224}}, {at::kFloat}})));
119+
testing::Values(PathAndInput({"tests/modules/resnet18_traced.jit.pt", {{1, 3, 224, 224}}, {at::kFloat}})));

tests/cpp/test_modules_as_engines.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,7 @@ TEST_P(CppAPITests, ModuleAsEngineIsClose) {
1414
jit_results.push_back(jit_results_ivalues.toTensor());
1515
auto trt_results = torch_tensorrt::tests::util::RunModuleForwardAsEngine(mod, inputs);
1616

17-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
18-
jit_results[0], trt_results[0].reshape_as(jit_results[0])));
17+
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(jit_results[0], trt_results[0].reshape_as(jit_results[0])));
1918
}
2019

2120
#ifndef DISABLE_TEST_IN_CI

tests/cpp/test_multiple_registered_engines.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,13 +56,13 @@ TEST(CppAPITest, CanRunMultipleEngines) {
5656
trt2_results.push_back(trt2_results_ivalues.toTensor());
5757

5858
for (size_t i = 0; i < trt1_results.size(); i++) {
59-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
60-
jit1_results[i], trt1_results[i].reshape_as(jit1_results[i])));
59+
ASSERT_TRUE(
60+
torch_tensorrt::tests::util::cosineSimEqual(jit1_results[i], trt1_results[i].reshape_as(jit1_results[i])));
6161
}
6262

6363
for (size_t i = 0; i < trt2_results.size(); i++) {
64-
ASSERT_TRUE(torch_tensorrt::tests::util::cosineSimEqual(
65-
jit2_results[i], trt2_results[i].reshape_as(jit2_results[i])));
64+
ASSERT_TRUE(
65+
torch_tensorrt::tests::util::cosineSimEqual(jit2_results[i], trt2_results[i].reshape_as(jit2_results[i])));
6666
}
6767
}
6868
#endif

tests/util/util.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ bool cosineSimEqual(const at::Tensor& computed_tensor, const at::Tensor& gt_tens
3232
std::ostringstream ss;
3333
ss << computed_tensor << std::endl << gt_tensor << std::endl;
3434
LOG_DEBUG(ss.str());
35-
if (computed_tensor.sum().item<float>() == 0.f || gt_tensor.sum().item<float>() == 0.f){
35+
if (computed_tensor.sum().item<float>() == 0.f || gt_tensor.sum().item<float>() == 0.f) {
3636
return almostEqual(computed_tensor, gt_tensor);
3737
} else {
3838
LOG_DEBUG(std::string("Cosine Similarity score: ") + std::to_string(cosine_sim.item<float>()));

0 commit comments

Comments
 (0)