<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<div class="moz-forward-container">Hi All,<br>
<br>
I have just committed changes reverting the default for the BCAL
simulation back to the original<br>
scheme. See the commit log below for details. This is in response
to a request from Paul who pointed<br>
that since the reconstruction is not able to handle the new scheme
yet, the changes made are quite<br>
disruptive to those using the reconstruction for analysis tasks.
As noted in the log, some annoying<br>
warning messages will be appearing at startup of the mcsmear and
any DANA program to remind<br>
us that this issue is still unresolved.<br>
<br>
This reversion is not a simple backing out of previous changes
as the mcsmear switch was moved<br>
from a compile time to a run time option. In other words, new code
was generated and with it, the<br>
possibility of new bugs. I think the likelihood of a problem is
small but if any new issues do develop, <br>
please let me know.<br>
<br>
Regards,<br>
-David<br>
<br>
-------- Original Message --------
<table class="moz-email-headers-table" border="0" cellpadding="0"
cellspacing="0">
<tbody>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Subject:
</th>
<td>r9772 - in trunk/sim-recon/src: libraries/BCAL
programs/Simulation/HDGeant programs/Simulation/mcsmear</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Date: </th>
<td>Wed, 10 Oct 2012 20:12:28 -0400</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">From: </th>
<td><a class="moz-txt-link-abbreviated" href="mailto:Hall-D.SVN.Repository@jlab.org">Hall-D.SVN.Repository@jlab.org</a></td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">To: </th>
<td><a class="moz-txt-link-abbreviated" href="mailto:richard.t.jones@uconn.edu">richard.t.jones@uconn.edu</a>, <a class="moz-txt-link-abbreviated" href="mailto:davidl@jlab.org">davidl@jlab.org</a></td>
</tr>
</tbody>
</table>
<br>
<br>
<pre>Author: davidl
Date: 2012-10-10 20:12:27 -0400 (Wed, 10 Oct 2012)
New Revision: 9772
Added:
trunk/sim-recon/src/programs/Simulation/mcsmear/smear_bcal_deprecated.cc
Modified:
trunk/sim-recon/src/libraries/BCAL/DBCALPoint.cc
trunk/sim-recon/src/libraries/BCAL/DBCALPoint_factory.cc
trunk/sim-recon/src/libraries/BCAL/DBCALPoint_factory.h
trunk/sim-recon/src/programs/Simulation/HDGeant/hitBCal.cc
trunk/sim-recon/src/programs/Simulation/mcsmear/mcsmear.cc
trunk/sim-recon/src/programs/Simulation/mcsmear/smear_bcal.cc
Log:
Reverting BCAL simulation back to original version as the default.
This is in response to Paul's request since the reconstruction is
not yet ready to handle the new scheme.
The change is not a complete reversal though:
1. Both types of BCAL data are now written out by default from the
BCAL in hdgeant.
2. Which method is used (new or old) in mcsmear is chosen by
command line parameter (-D) with the default being the old.
3. The DBCALPoint object no longer scales the signal amplitude
to convert from ADC counts to rough GeV since that is not
needed by the old scheme. This must be changed in the code
and recompiled to get the "new" behavior.
4. Both mcsmear and all DANA programs will print an annoying
warning message to remind users that they are using a deprecated
method.
</pre>
<br>
<br>
</div>
<br>
</body>
</html>