Skip to content

Commit ca9c7a3

Browse files
committed
Reduce DeepSSM tests from 4 to 2 configurations
Run only default and tl_net_fine_tune tests, which together cover all code paths (standard DeepSSM, TL-DeepSSM, and fine tuning). Cuts test time from ~3 minutes to ~90 seconds.
1 parent 36a6810 commit ca9c7a3

File tree

8 files changed

+12
-13
lines changed

8 files changed

+12
-13
lines changed

Python/DeepSSMUtilsPackage/DeepSSMUtils/image_utils.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@
22
import SimpleITK
33
import numpy as np
44

5-
def get_image_registration_transform(fixed_image_file, moving_image_file, transform_type='rigid'):
6-
# Prepare parameter map
5+
def get_image_registration_transform(fixed_image_file, moving_image_file, transform_type='rigid', max_iterations=1024):
6+
# Prepare parameter map
77
parameter_object = itk.ParameterObject.New()
88
parameter_map = parameter_object.GetDefaultParameterMap('rigid')
99
if transform_type == 'similarity':
1010
parameter_map['Transform'] = ['SimilarityTransform']
1111
elif transform_type == 'translation':
1212
parameter_map['Transform'] = ['TranslationTransform']
13-
parameter_map['MaximumNumberOfIterations'] = ['1024']
13+
parameter_map['MaximumNumberOfIterations'] = [str(max_iterations)]
1414
parameter_object.AddParameterMap(parameter_map)
1515

1616
# Load images

Testing/DeepSSMTests/DeepSSMTests.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,9 @@ void run_deepssm_test(const std::string& name) {
1111
}
1212

1313
//---------------------------------------------------------------------------
14+
// Run 2 configurations that cover all code paths:
15+
// - default: standard DeepSSM
16+
// - tl_net_fine_tune: TL-DeepSSM with fine tuning (covers both tl_net and fine_tune paths)
1417
TEST(DeepSSMTests, defaultTest) { run_deepssm_test("deepssm_default.sh"); }
1518

16-
TEST(DeepSSMTests, tlNetTest) { run_deepssm_test("deepssm_tl_net.sh"); }
17-
18-
TEST(DeepSSMTests, fineTuneTest) { run_deepssm_test("deepssm_fine_tune.sh"); }
19-
2019
TEST(DeepSSMTests, tlNetFineTuneTest) { run_deepssm_test("deepssm_tl_net_fine_tune.sh"); }

Testing/DeepSSMTests/deepssm_default.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,4 @@ rm -rf deepssm groomed *_particles
1414
shapeworks deepssm --name default.swproj --all
1515

1616
# Verify results
17-
python "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"
17+
python3 "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"

Testing/DeepSSMTests/deepssm_fine_tune.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,4 @@ rm -rf deepssm groomed *_particles
1414
shapeworks deepssm --name fine_tune.swproj --all
1515

1616
# Verify results
17-
python "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"
17+
python3 "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"

Testing/DeepSSMTests/deepssm_tl_net.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,4 @@ rm -rf deepssm groomed *_particles
1414
shapeworks deepssm --name tl_net.swproj --all
1515

1616
# Verify results
17-
python "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"
17+
python3 "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"

Testing/DeepSSMTests/deepssm_tl_net_fine_tune.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,4 @@ rm -rf deepssm groomed *_particles
1414
shapeworks deepssm --name tl_net_fine_tune.swproj --all
1515

1616
# Verify results
17-
python "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"
17+
python3 "${SCRIPT_DIR}/verify_deepssm_results.py" "${DATA}/deepssm/projects"

Testing/DeepSSMTests/run_exact_check.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ for config in $CONFIGS; do
3333
shapeworks deepssm --name ${config}.swproj --all
3434

3535
# Run exact check with config-specific file
36-
python "${SCRIPT_DIR}/verify_deepssm_results.py" . \
36+
python3 "${SCRIPT_DIR}/verify_deepssm_results.py" . \
3737
--exact_check "$MODE" \
3838
--baseline_file "exact_check_${config}.txt"
3939

Testing/DeepSSMTests/run_extended_tests.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ run_project() {
9999
verify_args="--expected 10 --tolerance 1.0"
100100
fi
101101

102-
python "${SCRIPT_DIR}/verify_deepssm_results.py" . $verify_args
102+
python3 "${SCRIPT_DIR}/verify_deepssm_results.py" . $verify_args
103103

104104
echo ""
105105
}

0 commit comments

Comments
 (0)