Skip to content
Snippets Groups Projects

Update to MicroBias settings

Merged Roel Aaij requested to merge raaij-MicroBias into master

Allow routing bit 48 to be configured from threshold settings and configure it from Calibration_VeloMicroBias.

This needn't go in the patch release, we can use it for the TCK and get it in next time.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Unable to load the diff
  • I am not sure if/how the new ODIN firmware will affect this filter. The first part can be replaced with ODIN_PASS(LHCb.ODIN.Physics), but I don't know about the second.

    What kind of data do we collect with this TCK?

  • Somehow my comment on the diff got lost. My only worry is the ODIN filter. I'm not sure if it is compatible with the new firmware, where the mantra is don't filter on trigger type. The first part can be replaced by ODIN_PASS(LHCb.ODIN.Physics) but I don't know what to do with TimingTrigger.

    How is this data taken / used, i.e. who knows what is the idea behind TimingTrigger?

    Edited by Rosen Matev
  • Otherwise, I'll include this in the release for the sake of consistency

  • Author Maintainer

    This is for the OT aging scan and the CCE scan for the Velo and ST. I think using NoBias triggers might be the way to go. I'll call Patrick and ask him.

  • Author Maintainer

    Federico will check and get back to me.

  • Author Maintainer

    There is a minbias L0 (0x1715) so we should get physics triggers. I think we can skip the TimingTrigger part. I'll update and commit.

  • Roel Aaij Added 3 commits:

    Added 3 commits:

  • LGTM, I'll merge it.

  • Rosen Matev Title changed from [WIP] Update to MicroBias settings to Update to MicroBias settings

    Title changed from [WIP] Update to MicroBias settings to Update to MicroBias settings

  • Reassigned to @liblhcb

  • @raaij, can you fix the conflict (in release.notes), please?

    It cannot be merged otherwise.

  • Roel Aaij Added 18 commits:

    Added 18 commits:

  • LHCb Librarian mentioned in commit 7cbb85ec

    mentioned in commit 7cbb85ec

  • LHCb Librarian Status changed to merged

    Status changed to merged

  • Please register or sign in to reply
    Loading