diff --git a/python/slsdet/detector.py b/python/slsdet/detector.py index 5a49442bc..6c9890efc 100755 --- a/python/slsdet/detector.py +++ b/python/slsdet/detector.py @@ -2172,8 +2172,8 @@ class Detector(CppDetectorApi): @property def vetofile(self): """ - [Gotthard2] Set veto reference for each 128 channels for specific chip. - The file should have 128 rows of gain index and 12 bit value in dec + [Gotthard2] Set veto reference for each 128 channels for specific chip. \n + The file should have 128 rows of gain index and 12 bit value in dec. Example --------- @@ -2198,6 +2198,16 @@ class Detector(CppDetectorApi): @property def vetophoton(self): + """ + [Gotthard2] Set veto reference for 128 channels for chip ichip according to reference file + and #photons and energy in keV. + Note + ---- + Arguments: (chip_index, n_photons, photon_energy, fname) + :getter: Not Implemented + Example + ------- + >>> d.vetophoton = (2, 24, 2560, '/tmp/bla.txt') raise NotImplementedError('vetofile is set only') @vetophoton.setter @@ -2776,7 +2786,7 @@ class Detector(CppDetectorApi): @property @element def v_b(self): - """[Ctb] Voltage supply a in mV.""" + """[Ctb] Voltage supply b in mV.""" return self.getDAC(dacIndex.V_POWER_B, True) @v_b.setter @@ -2786,7 +2796,7 @@ class Detector(CppDetectorApi): @property @element def v_c(self): - """[Ctb] Voltage supply a in mV.""" + """[Ctb] Voltage supply c in mV.""" return self.getDAC(dacIndex.V_POWER_C, True) @v_c.setter @@ -2796,7 +2806,7 @@ class Detector(CppDetectorApi): @property @element def v_d(self): - """[Ctb] Voltage supply a in mV.""" + """[Ctb] Voltage supply d in mV.""" return self.getDAC(dacIndex.V_POWER_D, True) @v_d.setter @@ -2806,7 +2816,11 @@ class Detector(CppDetectorApi): @property @element def v_io(self): - """[Ctb] Voltage supply a in mV.""" + """[Ctb] Voltage supply io in mV. Minimum 1200 mV. + Note + ---- + Must be the first power regulator to be set after fpga reset (on-board detector server start up). + """ return self.getDAC(dacIndex.V_POWER_IO, True) @v_io.setter diff --git a/slsDetectorSoftware/include/Detector.h b/slsDetectorSoftware/include/Detector.h index fcd166bd8..f84fab015 100644 --- a/slsDetectorSoftware/include/Detector.h +++ b/slsDetectorSoftware/include/Detector.h @@ -598,8 +598,14 @@ class Detector { * calculated (incremented by 1 if no 2nd interface)*/ void setDestinationUDPPort2(int port, int module_id = -1); + /** Reconfigures Detector with UDP destination. More for debugging as the + * configuration is done automatically when the detector has sufficient UDP + * details. */ void reconfigureUDPDestination(Positions pos = {}); + /** Validates that UDP configuration in the detector is valid. If not + * configured, it will throw with error message requesting missing udp + * information */ void validateUDPConfiguration(Positions pos = {}); Result printRxConfiguration(Positions pos = {}) const; @@ -1155,7 +1161,8 @@ class Detector { void setVetoReference(const int gainIndex, const int value, Positions pos = {}); - /** [Gotthard2] gain indices and adu values for each channel */ + /** [Gotthard2] Set veto reference for each 128 channels for specific chip. + * The file should have 128 rows of gain index and 12 bit value in dec"*/ void setVetoFile(const int chipIndex, const std::string &fname, Positions pos = {}); @@ -1549,15 +1556,12 @@ class Detector { void rebootController(Positions pos = {}); /** - * [Jungfrau][Gotthard][CTB][Moench] - * Advanced user Function! \n - * Updates the firmware, detector server and then reboots detector - * controller blackfin. \n - * [Mythen3][Gotthard2] Will still have old server starting up as the new - * server is not respawned \n - sname is name of detector server binary found on - * tftp folder of host pc hostname is name of pc to tftp from fname is - * programming file name + * Advanced user Function!\n [Jungfrau][Gotthard][CTB][Moench] Updates the + * firmware, detector server and then reboots detector controller blackfin. + * \n [Mythen3][Gotthard2] Will still have old server starting up as the new + * server is not respawned \n sname is name of detector server binary found + * on tftp folder of host pc \n hostname is name of pc to tftp from \n fname + * is programming file name */ void updateFirmwareAndServer(const std::string &sname, const std::string &hostname, diff --git a/slsDetectorSoftware/src/CmdProxy.cpp b/slsDetectorSoftware/src/CmdProxy.cpp index 9df334c54..9e2598097 100644 --- a/slsDetectorSoftware/src/CmdProxy.cpp +++ b/slsDetectorSoftware/src/CmdProxy.cpp @@ -1639,9 +1639,8 @@ std::string CmdProxy::VetoPhoton(int action) { os << "[ichip] [#photons] [energy in keV] [reference " "file]\n\t[Gotthard2] Set veto reference for 128 channels for " "chip ichip according to reference file and #photons and energy " - "in keV.\n\t" - << "[ichip] [output file]\n\t Get gain indices and veto reference " - "for 128 channels for chip ichip, saved to file." + "in keV.\n\t[ichip] [output file]\n\t Get gain indices and veto " + "reference for 128 channels for chip ichip, saved to file." << '\n'; } else if (action == defs::GET_ACTION) { if (args.size() != 2) { @@ -2476,9 +2475,13 @@ std::string CmdProxy::UpdateFirmwareAndDetectorServer(int action) { os << cmd << ' '; if (action == defs::HELP_ACTION) { os << "[server_name] [pc_host_name] " - "[fname.pof]\n\t[Jungfrau][Ctb][Moench] " - "Updates detector server via tftp from pc, updates firmware to " - "pof file and then reboots controller (blackfin)." + "[fname.pof]\n\t[Jungfrau][Gotthard][CTB][Moench] Updates the " + "firmware, detector server and then reboots detector controller " + "blackfin. \n\t[Mythen3][Gotthard2] Will still have old server " + "starting up as the new server is not respawned \n\tsname is " + "name of detector server binary found on tftp folder of host pc " + "\n\thostname is name of pc to tftp from \n\tfname is " + "programming file name" << '\n'; } else if (action == defs::GET_ACTION) { throw sls::RuntimeError("Cannot get");