Modified option StoreFirstMeasurements into StoredMeasurements

Modified option StoreFirstMeasurements into StoredMeasurements

! 2018-01-11 - Cristina Sanchez Gras, Patrick Koppenburg

• TupleToolTrackInfo : changed option StoreFirstMeasurement for StoredMeasurements. It changes the already existing X_TRACK_FirstMeasurement{X,Y,Z,QOverP,Tx,Ty} by X_TRACK_name_{X,Y,Z,QOverP,Tx,Ty}. It allows to get those track parameters at different parts of the detector, given by "name".
  • "name" can take many values: ClosestToBeam, FirstMeasurement, EndVelo, AtTT, AtT, BegRich1, EndRich1, BegRich2, EndRich2, Spd, Prs, BegECal, ECalShowerMax,MidECal, EndECal, BegHCal, MidHCal, EndHCal, Muon, LastMeasurement, Vertex, V0Vertex

mentioned in merge request !459 (closed)

Can we reach a conclusion on this? Cristina is now able to simulate the whole Gauss->Boole->Brunel->DaVinci at Nikhef, but I'd like this to be done centrally.

@pkoppenb, we do what we can ;-). The PR is now here and if all goes fine it will get merged tomorrow after a check in the nightlies - standard procedure.

But you won't have this in production until there is a release, which won't happen tomorrow at all. With a bit of luck I will manage to get this in together with the software stack release for the ongoing stripping campaign. I hope you are not suggesting to make a release for only this update …

Thanks for your understanding.

Gitlab is playing tricks on me. This comment was not supposed to go in this MR. Apologies.

• [2019-01-31 00:13] Validation started with lhcb-run2-patches#276

One important correction to be made. Then I can merge this. Thanks.

added 1 commit

• 403e7532 - Put back protection for nDoF>=256

Compare with previous version

resolved all discussions

mentioned in commit b20db38b

merged

@crsanche, one more thing and we are done: I did not manage to cherry-pick this MR to master, because of other incompatible changes to the files, I guess. Can you kindly prepare a similar MR to master? Then you're done :-).

Many thanks!

@erodrigu Yes, sure. I'll try my best to fix the incompatibilities!

The easiest will be for you to copy over your 2 files to master and edit there, I think ...

Yes, I'll do that, I didn't explain myself correctly In the previous MR I made to master, there were some incompatibilities with ITrackExtrapolator, but I didn't manage to fix it back then. I'll try harder this time I meant.

mentioned in merge request !466 (merged)