Merge pull request #2429 from SCIInstitute/fix-2428-groom-fixed-alignment

akenmorris · web-flow · commit 81c093a8baee · 2025-09-25T14:00:24.000-06:00
Fix #2428 - Groom: Fixed domains should not clear transforms on grooming
diff --git a/Libs/Groom/Groom.cpp b/Libs/Groom/Groom.cpp
@@ -44,9 +44,11 @@ bool Groom::run() {
     throw std::invalid_argument("No subjects to groom");
   }
 
-  // clear alignment transforms
+  // clear alignment transforms (except fixed subjects)
   for (auto& subject : subjects) {
-    subject->set_groomed_transforms({});
+    if (!subject->is_fixed()) {
+      subject->set_groomed_transforms({});
+    }
   }
 
   total_ops_ = get_total_ops();

Original file line number	Diff line number	Diff line change
`@@ -44,9 +44,11 @@ bool Groom::run() {`
`44`	`44`	`throw std::invalid_argument("No subjects to groom");`
`45`	`45`	`}`
`46`	`46`
`47`		`- // clear alignment transforms`
	`47`	`+ // clear alignment transforms (except fixed subjects)`
`48`	`48`	`for (auto& subject : subjects) {`
`49`		`- subject->set_groomed_transforms({});`
	`49`	`+ if (!subject->is_fixed()) {`
	`50`	`+ subject->set_groomed_transforms({});`
	`51`	`+ }`
`50`	`52`	`}`
`51`	`53`
`52`	`54`	`total_ops_ = get_total_ops();`