Skip to content
New issue

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

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

Already on GitHub? # to your account

Add digitizer codes. #24

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ABlairJamieson
Copy link
Contributor

First attempt at adding a digitizer code to apply to truth hits. Many parameters are available in the Digitizer class to tune the digitizer to the one chosen.

The code runs fairly slow, so some work to optimize this are likely needed.

@pdeperio pdeperio requested a review from nickwp March 17, 2021 00:31
@pdeperio
Copy link
Collaborator

Does this belong here? How exactly does it interact with the WatChMaL framework?

In principle, this should be part of simulation (WCSim), so maybe we should make a new repo somewhere (IWCD, WCTE, HK GitHub orgs?)

@nickwp
Copy link
Collaborator

nickwp commented Jul 19, 2021

I think the plan now is to put this type of implementation into the MDT software by @akutsuR (then can be integrated into watchmal in future, or just used in MDT, as required).
@ABlairJamieson is it okay if we close this PR, with the intention of integrating it into MDT in future?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants