<div dir="ltr">Hi Elton,<div><br></div><div>Thanks for your close reading of the document,  I&#39;m not going to be able to attend tomorrow&#39;s Calorimetry meeting due to a conflict, but I&#39;d like to make a couple comments to help clarify the situation (at least as I understand it).  I&#39;ll restrict myself to discussing Table 2, since as you said the information in the first table should be stored in the online databases which are still under development.</div>


<div><br></div><div>You&#39;re right that in the end we want a set of constants that represent our best understanding to be applied during reconstruction (e.g. ADC gains, pedestals, and time offsets).  My understanding is that it would be useful to keep values derived from specific data sets in the database as intermediate steps, to allow for communication between different calibration programs.  For example, maybe you have one program that determines ADC constants from cosmic data, another one that determines them from pi0 and eta decays, and then a final one that combines this information obtain the final set of constants. In this case, the ADC_physics_XXXX would be the per-channel values determined from the pi0 and eta decays. Of course, the details of what intermediate information to save should depend on the details of the procedures...</div>
<div><br></div><div>I&#39;d suggest that since the CCDB allows arbitrarily complex pathnames for its tables, that any of this intermediate information be kept in subdirectories (e.g. &quot;/BCAL/cosmics/ADC_gains&quot;), to reduce confusion.</div>
<div><br></div><div><br></div><div>Cheers,<br>Sean</div><div><br></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 28, 2014 at 11:39 AM, Elton Smith <span dir="ltr">&lt;<a href="mailto:elton@jlab.org" target="_blank">elton@jlab.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Zisis,<br>
<br>
Here are some belated comments to your document GlueX-doc-2379. I notice<br>
I made my comments on the version on Dec 17 and there is an updated<br>
version. In a  cursory glance I did not the changes, but I could be<br>
providing redundant suggestions.<br>
<br>
<br>
General: It is good to have the GlueX-doc number at the top. My hard<br>
copy lacked this and it took me a while to find where I got the document.<br>
<br>
Th first table (conditions parameters) contains very similar information<br>
to what may be included in online data bases. These are still under<br>
discussion, so the two need to be designed together so that they<br>
complement each other and minimize duplication. For example, the SiPM<br>
bias voltages will be recorded and monitored by EPICS and I don&#39;t know<br>
how this information will be needed beyond that.<br>
<br>
LED_cntl_clock, LED_cntl_width, LED_bias<br>
There are a total of 32 independent pulsers (16 upstream, 16<br>
downstream). Each of 4 pulses are duplicated 12 times and delivered to<br>
quadrants, up or down.<br>
I assume the clock is the rate at which the pulser is pulsed.<br>
<br>
These next two are clearly DAQ settings and would be part of the<br>
configuration files<br>
ADC_threshold. Is this the same as the pedestal subtracted for the<br>
trigger? Is it a sparcification threshold?<br>
TDC_threshold. Is this the discriminator threshold?<br>
<br>
The next will be recorded and monitored by EPICs<br>
SPM_bias_hv<br>
SPM_bias_lv. There are a total of 48 independent channels (24 +5V, 24<br>
-5V). Each channel serves 4 readout assemblies on one side.<br>
SPM_crate<br>
N2_flow. At the moment we do not monitor the N2 flow, they are visual<br>
flow meters. However, we do monitor temp and humidity inside the readout<br>
assemblies<br>
<br>
This information should be set in a configuration file<br>
COOLING_temp - I assume this is the set point for the chiller? This<br>
would be the temperature of the water in the cooling loop<br>
<br>
The following information should be monitored and controlled from EPICS<br>
SPM_temp - There is one RTD temperature probe on each cooling plate,<br>
i.e. 96.<br>
Readout assembly temperature and humidity. There are 8 probes (4<br>
upstream, 4 downstream) in the air (N2) volume of the readout<br>
assemblies. They measure temperature and humidity, so the dew point can<br>
be calculated.<br>
HALLD_humidity - There is only discussion at the moment to add a probe<br>
to determine the general conditions of the hall.<br>
<br>
Table 2 (Calibration constants)<br>
<br>
I assume we need an offset and gain for each ADC channel is related to<br>
the ADC_...._position and width information.   While I agree that there<br>
may be different methods of determining the gain (cosmics, pulser,<br>
physics, etc), I think we probably want to store somewhere the &quot;best<br>
estimate&quot; of the actual gain information for each ADC channel. The<br>
procedure for determining this value is yet to be determined and may use<br>
several of the inputs so far mentioned.<br>
<br>
ADC_gain 1536 channels<br>
ADC_offset: 1536 channels<br>
ADC_time_offset: 1152 channels<br>
<br>
For the ADC_physics<br>
What is meant to the ADC_physics_position or TDC_physics_position, etc<br>
values? The intent is to use pi0 and eta production to extract the<br>
information. However, these will not yield a single peak, but rather pi0<br>
peaks will be determined using combination of two showers (each shower<br>
with multiple ADC channel). From this a  procedure will have to be<br>
developed to determine the gains of each channel.<br>
<br>
For the TDC_physics_<br>
What would be used for the source of this information?<br>
<br>
Thanks, Elton.<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Elton Smith<br>
Jefferson Lab MS 12H5<br>
12000 Jefferson Ave STE 16<br>
Newport News, VA 23606<br>
<a href="tel:%28757%29%20269-7625" value="+17572697625">(757) 269-7625</a><br>
<a href="tel:%28757%29%20269-6331" value="+17572696331">(757) 269-6331</a> fax<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>Dr. Sean Dobbs<br>Department of Physics &amp; Astronomy <br>Northwestern University<br>phone: 847-467-2826
</div>