sign change of value impact

Interactive-Intelligent-Systems · Feb 11, 2024 · 36ad8c7 · 36ad8c7
1 parent 3688e21
commit 36ad8c7
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/main/java/diarg/values/ValueBasedAgreementScenario.java b/src/main/java/diarg/values/ValueBasedAgreementScenario.java
@@ -163,8 +163,7 @@ public double determineMedianAgreement() {
     public double determineSatisfactionImpact(int index1, int index2, Value value) {
         ValueBasedAgreementScenario counterfactualVBAScenario = this.clone();
         counterfactualVBAScenario.removeValue(value);
-        return counterfactualVBAScenario.determineSatisfaction(index1, index2) -
-                determineSatisfaction(index1, index2);
+        return determineSatisfaction(index1, index2) - counterfactualVBAScenario.determineSatisfaction(index1, index2);
     }
 
     /**
@@ -176,7 +175,7 @@ public double determineSatisfactionImpact(int index1, int index2, Value value) {
     public double determineMinimalAgreementImpact(Value value) {
         ValueBasedAgreementScenario counterfactualVBAScenario = this.clone();
         counterfactualVBAScenario.removeValue(value);
-        return counterfactualVBAScenario.determineMinimalAgreement() - determineMinimalAgreement();
+        return determineMinimalAgreement() - counterfactualVBAScenario.determineMinimalAgreement();
     }
 
     /**
@@ -188,7 +187,7 @@ public double determineMinimalAgreementImpact(Value value) {
     public double determineMeanAgreementImpact(Value value) {
         ValueBasedAgreementScenario counterfactualVBAScenario = this.clone();
         counterfactualVBAScenario.removeValue(value);
-        return counterfactualVBAScenario.determineMeanAgreement() - determineMeanAgreement();
+        return determineMeanAgreement() - counterfactualVBAScenario.determineMeanAgreement();
     }
 
     /**
@@ -200,7 +199,7 @@ public double determineMeanAgreementImpact(Value value) {
     public double determineMedianAgreementImpact(Value value) {
         ValueBasedAgreementScenario counterfactualVBAScenario = this.clone();
         counterfactualVBAScenario.removeValue(value);
-        return counterfactualVBAScenario.determineMedianAgreement() - determineMedianAgreement();
+        return determineMedianAgreement() - counterfactualVBAScenario.determineMedianAgreement();
     }