static std::vector
with
a pointer to same, initialized to zero, and modifying code to allocate
(once) with new
at an appropriate time. The memory corruption
in AcdUtil::AcdCalib was fixed by a similar technique. To keep gcc 3.4
(and perhaps other, less articulate compilers) happy,
Do not declare potentially variable length objects,
such
as vectors, to be static
.
The amount of static memory required should be knowable at compile time,
without having to run executable code.
(Richard) The new release was used for a muon run and sample day test, demonstrating that there were still outstanding issues, chief among them the efficiency of ACD, which was only 35%! Another unpleasant characteristic was bifurcated values per run for latitude and longitude. It was looking bleak for the home team. However, late yesterday Eric C. determined the ACD problems were due to inconsistent calibrations. Job options need to change for the new ACD code. He will put together an appropriate set of calibrations, we will adjust job options and try again. The latitude/longitude strangeness is probably something we can live with for now, perhaps attributable to the short runs. The first event or three have one pair of values, the rest a different pair. (Toby) How does one know what the right job options should be? (Richard) We need to clean up job options again, rationalizing use of includes [hearty agreement from all sides].
RandomSvc Default is still to set seed once per event, rather than once per run as agreed at a previous meeting. (Toby) Let's change the default. (Heather) Yes, let's. (Richard) will make the change in job options for the upcoming test runs to use the new scheme.
CAL status Due to issues with EbfWriter, we're not yet fully using Zach's Cal upgrades; in particular we're still running CalDigiAlg before TriggerAlg.
EBF data direct (Heather) [sent me the following, for which much thanks!]
..concerning putting the EBF data from the evt files onto the TDS, rather than having EbfWriter recompute the EBF data blob and putting it on the TDS for the OBF to use, as it does now. This issue came out of the OktoberTest, where it was determined that the GCR events were getting dropped due to some filtering done by EbfWriter - not really a deficiency in EbfWriter, as it was never written to handle processing of "real" data (even in this case, where the real data was simulated..but that's where my head starts to spin and I best not think too deeply about that). So the solution first suggested by Bryson was that we take the EBF data that already exists in the evt files, and just put that on the TDS. So I've implemented that. Unfortunately, I haven't quite got the exact blob I need. There must be extra bytes or something, since when OBF gets my blob, it fails and complains somewhere in the FSW. I'm looking into it.
Composite Event Lists (Heather) CEL will go into HEAD soon, after David Chamont has had a chance to respond to her query for a good set of tags to use.
Interleave(Tracy) He's restructuring, inventing new GaudiTools, to use CEL and to support possible Level 0 interleave. It's all checked into CVS, awaiting the promotion of CEL to HEAD. (Toby) suggests Interleave should keep statistics on how many times each group of events is selected. (Tracy) has been using Leon's new job options arrangement; it seems fine. He would like to move the Interleave-specific job options file out of the Gleam package to Interleave.
See the blog for late-breaking news.
(Richard) Expect Rob to call a meeting next week to discuss MOOT aspects of Filter Configuration and related issues.
warning: passing `double' for converting 1 of `int abs(int)'but some packages have so many warnings (in v13r4, rhel 4 build, CRflux is the leader, with 254) that it is impossibly tedious to read through them all. Most of them are benign and can be eliminated with proper recoding. It's just as important to keep test programs up to date so that failures are meaningful. (Jim) Warnings can come from include files belonging to codebases not under our control. This was the case for Likelihood and Xerces. (Joanne) Many warnings from Xerces code disappeared when we moved from 2.6 to 2.7 [in fact currently the Likelihood build has no Xerces-related warnings], but this kind of thing certainly can be a problem. (Toby) This can be handled by a filter configured to look for certain known patterns.
previous | minutes index | next |