regarding conflicts with new volumes in GEANT etc...

From: Adrian T Sindile (asindile@cisunix.unh.edu)
Date: Mon Oct 22 2001 - 19:49:42 EDT


> At any rate, as it currently stands, I can't get the newest version of
> "blast.geom" to work in TBLGeometry. However, if I use the older
> version of "blast.geom", then my reconstruction won't take into account
> these new volumes already added to GEANT. Any suggestions?

Hi, Aaron, everybody!
I was out of town (and with no e-mail access) last week, so I can only
answer now.
I added those volumes to GEANT for the energy loss study. The version I
ended up with worked perfectly for me, however, I did not need to change
any class (TBLGeometry or whatever), so the class should be unchanged.

> Orrrrrrrrrrrrrrrrrrrr... I can just comment out the "new" volume
> parameters from the file "blast.geom" and from the corresponding
> xx_geom.f files. However, I don't know which xx_geom.f files have been
> updated for these new volumes.

I thought the whole idea of CVS was that people work independently and
merge the changes, so that we end up with realistic stuff. Once something
is checked in, whoever has something new is responsible to make everything
work again when he submits his changes (and nothing more), WITHOUT getting
rid of the previous (useful) changes.
When I added the new volumes, I checked everything and it worked, and
nothing else needed to be done from my point of view...
So, Aaron, as Maurik said, please do not commit versions that do not work
with my previous changes, you can always check out older versions and just
test them, until you make everything work with the latest release.
I had similar problems when I made my changes, as Maurik was quicker at
that time and made chenges of his own, however, it was my responsibility
then to make sure I do not waste his work!
Finally, thanks, Tim, for taking care of that class and making it
compatible with the volumes I added (I have to stress, I did not need to
modify it then).
Sorry for not responding faster!

Adrian



This archive was generated by hypermail 2.1.2 : Mon Feb 24 2014 - 14:07:28 EST