I know that we "agreed" that BRAT is to be frozen in the pro directory. Do we really want it frozen with the important mistake Djamel found???? I will not do anything until I hear otherwise. I personally would vote to put in the correct BrMagnetVolume as I think that having the correct current set when you are trying to match tracks is more important than freezing the volume, but I await the advice of the movers and shakers. Kris P. S. It was my coding that was in error. It was obviously a cut and paste whose changes did not get carried far enough. SORRY! Djamel Ouerdane <ouerdane@nbi.dk> said: > Hi all, > > After having noticed some inconsistency in momenta from geant tracks > and reconstructed tracks, I checked the class BrMagnetVolume.cxx and > found out that the method FillMagnetParameters was wrong. > > All magnet currents and polarities were set with the one from D1 !! > Now, I don't know if this very bad mistake really propagates if you set > the current and pol. in your program after you get the magnet volume from > the geo manager. Anyway, it is extremely bad. It's now fixed but I > haven't tagged brat. We probably should. > > Djam > > -- > Djamel Ouerdane ------------------------------------------o > | Niels Bohr Institute | Home: | > | Blegdamsvej 17, DK-2100 Ø | Jagtvej 141 2D, | > | Fax: +45 35 32 50 16 | DK-2200 Copenhagen N | > | Tel: +45 35 32 52 69 | +45 35 86 19 74 | > | http://www.nbi.dk/~ouerdane | > | ouerdane@nbi.dk | > o---------------------------------------------------------o > --
This archive was generated by hypermail 2b30 : Wed Jun 19 2002 - 22:19:14 EDT