Remove calls to zeroOffset() in Longitudinal.cc to stop SXR loop from…

… snapping to HXR value when the beam trips off
slaclab · Aug 22, 2020 · a9a7cbc · a9a7cbc
1 parent 5448b7f
commit a9a7cbc
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 7 deletions.
diff --git a/RELEASE_NOTES b/RELEASE_NOTES
@@ -11,6 +11,11 @@
 #               2018, W. Williams        (ernesto)
 #      
 #==============================================================
+R3.2.1:   25-August-2020 J. Mock (jmock)
+    * Remove calls to zeroOffset() in Longitudinal.cc to stop SXR loop from snapping
+        to HXR value when the beam trips off
+    * Also comment out _referenceOffset = 0 in ActuatorDevice.cc::clear() for same reason
+
 R3.2.0:   25-August-2020 R.Reno (rreno)
     * Add ability to select BY1 energy source at IOC startup
     * Fixed bug where error handling code for an FCOM channel creation failure would never run

diff --git a/fastFeedbackApp/src/framework/ActuatorDevice.cc b/fastFeedbackApp/src/framework/ActuatorDevice.cc
@@ -261,7 +261,7 @@ void ActuatorDevice::clear() {
     Device::clear();
     _nextWrite = 0;
     _droppedPoints = 0;
-    _referenceOffset = 0;
+    //_referenceOffset = 0;
     Log::getInstance() << Log::flagOffset << Log::dpInfo
 		       << "ActuatorDevice::clear() _referenceOffset=0"
 		       << Log::dp;

diff --git a/fastFeedbackApp/src/framework/Longitudinal.cc b/fastFeedbackApp/src/framework/Longitudinal.cc
@@ -1171,7 +1171,8 @@ int Longitudinal::selectStates() {
 		_latestMeasurementsSum[0] = 0;
 
 		// Zero actuator offsets!
-		actuatorDevice->zeroOffset();
+    //2020-08-21 J. Mock remove this for dual energy operation
+		//actuatorDevice->zeroOffset();
 	}
 	stateDevice->setUsedBy(stateUsed);
 	actuatorDevice->setUsedBy(stateUsed);
@@ -1252,7 +1253,8 @@ int Longitudinal::selectStates() {
 		_latestMeasurementsSum[1] = 0;
 
 		// Zero actuator offsets!
-		actuatorDevice->zeroOffset();
+    //2020-08-21 J. Mock remove this for dual energy operation
+		//actuatorDevice->zeroOffset();
 	}
 
 	stateDevice->setUsedBy(stateUsed);
@@ -1338,7 +1340,8 @@ int Longitudinal::selectStates() {
 		_latestMeasurementsSum[2] = 0;
 
 		// Zero actuator offsets!
-		actuatorDevice->zeroOffset();
+    //2020-08-21 J. Mock remove this for dual energy operation
+		//actuatorDevice->zeroOffset();
 	}
 	stateDevice->setUsedBy(stateUsed);
 	actuatorDevice->setUsedBy(stateUsed);
@@ -1417,7 +1420,8 @@ int Longitudinal::selectStates() {
 		_latestMeasurementsSum[3] = 0;
 
 		// Zero actuator offsets!
-		actuatorDevice->zeroOffset();
+    //2020-08-21 J. Mock remove this for dual energy operation
+		//actuatorDevice->zeroOffset();
 	}
 	stateDevice->setUsedBy(stateUsed);
 	actuatorDevice->setUsedBy(stateUsed);
@@ -1498,7 +1502,8 @@ int Longitudinal::selectStates() {
 		_latestMeasurementsSum[4] = 0;
 
 		// Zero actuator offsets!
-		actuatorDevice->zeroOffset();
+    //2020-08-21 J. Mock remove this for dual energy operation
+		//actuatorDevice->zeroOffset();
 	}
 	stateDevice->setUsedBy(stateUsed);
 	actuatorDevice->setUsedBy(stateUsed);
@@ -1652,7 +1657,8 @@ int Longitudinal::selectStates() {
 		_latestMeasurementsSum[5] = 0;
 
 		// Zero actuator offsets!
-		actuatorDevice->zeroOffset();
+    //2020-08-21 J. Mock remove this for dual energy operation
+		//actuatorDevice->zeroOffset();
 	}
 	stateDevice->setUsedBy(stateUsed);
 	actuatorDevice->setUsedBy(stateUsed);