JeffersonLab / sim-recon

Simulation and Reconstruction for GlueX
9 stars 14 forks source link

Davidl cdc pulse peak #994

Closed faustus123 closed 6 years ago

faustus123 commented 6 years ago

Create DCDCDigiHit objects from Df125FDCPulse objects if they come from CDC channels. Added pulse_peak member to DCDCDigiHit

gluex commented 6 years ago

Test status for this pull request: SUCCESS

Summary: /work/halld/pull_request_test/sim-recon^davidl_CDC_pulse_peak/tests/summary.txt Logs: /work/halld/pull_request_test/sim-recon^davidl_CDC_pulse_peak/tests/log

Build log: /work/halld/pull_request_test/sim-recon^davidl_CDC_pulse_peak/make_davidl_CDC_pulse_peak.log Build report: /work/halld/pull_request_test/sim-recon^davidl_CDC_pulse_peak/report_davidl_CDC_pulse_peak.txt Location of build: /work/halld/pull_request_test/sim-recon^davidl_CDC_pulse_peak

nsjarvis commented 6 years ago

libraries/CDC/DCDCHit_factory.cc still needs to be changed otherwise hit->q will be 0 when there is no cdcpulse

On 5 December 2017 at 14:24, zihlmann notifications@github.com wrote:

Merged #994 https://github.com/JeffersonLab/sim-recon/pull/994.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/JeffersonLab/sim-recon/pull/994#event-1373355278, or mute the thread https://github.com/notifications/unsubscribe-auth/AO7hVq8xvEjDfWR8e23ObJnhqRgTgiQXks5s9ZhhgaJpZM4Q2kHk .