-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Description
ADPilatus/pilatusApp/src/pilatusDetector.cpp
Line 1360 in 5a649ce
| setDoubleParam(PilatusThTemp0, temp); |
When reading detector temp and humidity, there is an entry for Channel 3 that is unlikely to exist. And worse, if it exists it overrides Channel 0:
if ((substr = strstr(this->fromCamserver, "Channel 3")) != NULL) {
sscanf(substr, "Channel 3: Temperature = %fC, Rel. Humidity = %f", &temp, &humid);
setDoubleParam(PilatusThTemp0, temp);
setDoubleParam(PilatusThHumid0, humid);
}
IMHO I believe this piece of code should be removed to avoid confusion.
However, I ignore if there is a case of Pilatus systems where the channels are numbered [1..3] instead of [0..2] by CamServer -- if that's the case then it should be properly documented, and set ADTemperature as well.
Thanks,
Armando
Metadata
Metadata
Assignees
Labels
No labels