<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Hi Mike,</div><div class=""><br class=""></div>I think that you should just push to the master branch. It’s a relatively small change and it’s not going to break anything. I think we should bit the bullet and get this started so that all the downstream calibrations etc. can be done done.<div class=""><br class=""><div class="">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Best,<br class="">Mark Macrae Dalton</div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""></div><br class="Apple-interchange-newline">
</div>
<br class=""><div><blockquote type="cite" class=""><div class="">On Mar 29, 2016, at 1:55 PM, Michael Staib <<a href="mailto:mstaib@andrew.cmu.edu" class="">mstaib@andrew.cmu.edu</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi All,<br class=""><br class="">I have pushed a branch to Github, called f250_NSA_NSB, that grabs the correct values for NSA and NSB from the flash registers when the BOR record is available. My thoughts are to leave the code on a branch for now until calibration constants are ready for the fixed values. <br class=""><br class="">In case you are unaware from previous meetings, it was found that NSB = 0 is an invalid setting on the current version of the f250. For the BCAL, the configuration file had NSB = 0 and NSA=26, FCAL, NSB = 0 and NSA=15, so we were subtracting off 26 and 15 samples of pedestal, respectively. In reality, on the flash NSB was being forced to 1. The new code will correctly subtract 27 and 16 samples from the BCAL/FCAL by grabbing these values from the registers. This will obviously cause some small shift in the energy calibrations.<br class=""><br class="">If you think I should just pull the fix into the master branch, let me know.<br class=""><br class="">--<br class="">Michael Staib<br class="">Graduate Student, Dept. of Physics<br class="">Carnegie Mellon University<br class=""><a href="mailto:mstaib@cmu.edu" class="">mstaib@cmu.edu</a><br class="">phone: 412-268-2983<br class=""><br class=""><br class=""><br class=""><br class="">_______________________________________________<br class="">Halld-cal mailing list<br class="">Halld-cal@jlab.org<br class="">https://mailman.jlab.org/mailman/listinfo/halld-cal<br class=""></div></div></blockquote></div><br class=""></div></body></html>