#98 changed order at Neighborhood::random

IDSIA · Oct 12, 2021 · 0774f68 · 0774f68
1 parent fdfad84
commit 0774f68
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/main/java/ch/idsia/crema/inference/approxlp1/Neighbourhood.java b/src/main/java/ch/idsia/crema/inference/approxlp1/Neighbourhood.java
@@ -62,10 +62,11 @@ public Solution move(Solution from, Move doing) {
 	private BayesianFactor random(GenericFactor factor) {
 		if (factor instanceof ExtensiveLinearFactor) {
 			return new ExtensiveLinearToRandomBayesian().apply((ExtensiveLinearFactor<?>) factor);
+		// TODO: issue #98, make this work without non-negative constraints
+		//} else if (factor instanceof SeparateHalfspaceFactor) {
+		//	return new HalfspaceToRandomBayesianFactor().apply((SeparateHalfspaceFactor) factor);
 		} else if (factor instanceof SeparateLinearFactor) {
 			return new SeparateLinearToRandomBayesian().apply((SeparateLinearFactor<?>) factor);
-		} else if (factor instanceof SeparateHalfspaceFactor) {
-			return new HalfspaceToRandomBayesianFactor().apply((SeparateHalfspaceFactor) factor);
 		} else if (factor instanceof BayesianFactor) {
 			return (BayesianFactor) factor;
 		}