Decoding FT v7 in Allen and fixes to CPU v7
Requires LHCb!3572 (merged) and Detector!202 (merged)
- Implements scifi v7 decoding in Allen.
- Changes the format of previous scifi geometries, including the geometry version. Hence, all previously dumped geometry versions of Allen become incompatible, and new binary files are required.
- Implements fixes in LHCb where the bank reordering between A side and C side was not properly implemented and led to a crash in Boole. This is now corrected.
- Implements several safeguards to reduce seg faults in case the link map is wrong (this is still not 100% done)
- Related: hardcoded
station-1
statements spell difficult-to-debug disaster when an invalid channel is encountered. In general the station = [1,2,3] and station = [0,1,2] conventions need to be spelled out much better. - This will be for @jheuel MR on expliciting our conventions
- Related: hardcoded
Edited by Daniel Hugo Campora Perez
Merge request reports
Activity
added RTA label
- Resolved by Louis Henry
- Resolved by Louis Henry
- Resolved by Louis Henry
- Resolved by Dorothea Vom Bruch
- Resolved by Louis Henry
- Resolved by Louis Henry
- Resolved by Louis Henry
added 1 commit
- 7dc584c6 - Fixed v7 decoding with a hack to test it on v6 samples
mentioned in merge request Moore!1568 (closed)
added 9 commits
-
7dc584c6...74abe94b - 5 commits from branch
master
- eda4e384 - v7 decoding in Allen which passes the mdf_input execution
- 998f0c9d - Removed test message
- 411a3827 - Fixed formatting
- 0ed0c436 - Fixed v7 decoding with a hack to test it on v6 samples
Toggle commit list-
7dc584c6...74abe94b - 5 commits from branch
added 88 commits
-
b7c7e11b...5cef75c2 - 87 commits from branch
master
- 070b8506 - Merge with master
-
b7c7e11b...5cef75c2 - 87 commits from branch
added 1 commit
- a103c2a0 - Started to make the code more error-resilient
added 8 commits
-
a103c2a0...f32f438e - 5 commits from branch
master
- d1b1d76a - Merge with master
- c370cd6f - Removed hacks to force v7 on v6
- 868e7617 - Removing unwanted file
Toggle commit list-
a103c2a0...f32f438e - 5 commits from branch
Please register or sign in to reply