Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update the (pre|post)Source(Lumi|Run) signals #21872

Merged
merged 2 commits into from
Jan 28, 2018
Merged

Update the (pre|post)Source(Lumi|Run) signals #21872

merged 2 commits into from
Jan 28, 2018

Commits on Jan 17, 2018

  1. Update the (pre|post)Source(Lumi|Run) signals 

    Update the signals to pass
  - the `LuminosityBlockIndex` to the `(pre|post)SourceLumi` signals;
  - the `RunIndex` to the `(pre|post)SourceRun` signals.

Update the client code to ignore the new parameters.
    @fwyzard
    fwyzard committed Jan 17, 2018
    Configuration menu
    Copy the full SHA
    fa3092b View commit details
    Browse the repository at this point in the history

  2. FastTimerService: make use of the updated signals 

    Take advantage of the updated signatures for the `(pre|post)Source(Lumi|Run)`
signals to account for the time spent in the Source transitions.
    @fwyzard
    fwyzard committed Jan 17, 2018
    Configuration menu
    Copy the full SHA
    5b8e8d2 View commit details
    Browse the repository at this point in the history