Skip to content
Snippets Groups Projects

Introduce possibility to simulate time-based with constant time between events. Refs #1792.

Merged Volker Friese requested to merge v.friese/cbmroot:sim_constant_deltat into master
All threads resolved!

In time-based simulations, the time between two subsequent events is sampled from an exponential distribution (Poisson process). For debug reasons, it was requested to have the possibility to generate constant time intervals between events - see #1792. This MR introduces the required functionality.

For this, a new enum cbm::sim::TimDist was introduced, which can have the two value Poisson and Uniform. It has to be specified when specifying an input to the digitization run, as can be seen in run_digi.C:

CbmDigitization run;
cbm::sim::TimeDist timeDist = cbm::sim::TimeDist::Uniform;
run.AddInput(0, evntFile, timeDist, eventRate);

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
  • Volker Friese added 1 commit

    added 1 commit

    • 677c580f - Add Defs.h, forgotten in the previous commit

    Compare with previous version

  • Volker Friese added 1 commit

    added 1 commit

    • 43fafe68 - License in Defs.h; medmber initialisation in CbmDigitization.

    Compare with previous version

  • Dear @f.uhlig, @p.-a.loizeau, @c.muentz, @m.deveaux, @v.friese,

    you have been identified as code owner of at least one file which was changed with this merge request.

    Please check the changes and approve them or request changes.

  • Florian Uhlig resolved all threads

    resolved all threads

  • Florian Uhlig added 27 commits

    added 27 commits

    • 43fafe68...55cdf48c - 23 commits from branch computing:master
    • 46f20daf - Introduce possibility to simulate time-based with constant time between events. Refs #1792.
    • c14af8ae - Install header-only file
    • 8c97fdc1 - Add Defs.h, forgotten in the previous commit
    • 8227fc35 - License in Defs.h; medmber initialisation in CbmDigitization.

    Compare with previous version

  • Since @v.singhal approved this MR (added :thumbsup:) I will merge it.

  • Florian Uhlig enabled an automatic merge when the pipeline for 8227fc35 succeeds

    enabled an automatic merge when the pipeline for 8227fc35 succeeds

  • merged

  • Volker Friese mentioned in merge request !1270 (merged)

    mentioned in merge request !1270 (merged)

  • Martin Beyer mentioned in merge request !2041 (merged)

    mentioned in merge request !2041 (merged)

  • Please register or sign in to reply
    Loading