eic / eic/eic-smear

Double32_t has insufficient precision

Open
#9 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
2
Forks
13
PR merge metrics
No merged PRs in 30d

Description

Reported by Wojtek Slominski. This type is used by most if not all MC classes and offers 7-10 significant digits, and getting to 9-10 would already require changes of the form
Double32_t E; //[0,10000]
where the comment is important.

An example from rapgap:
(in MC notation: px, py, pz, E, M):
Generator output TXT file:
1.472406611 -1.440001479 -17.51050923 17.631208672 0.000510000
ROOT tree track filled by BuildTree:
1.472406611 -1.440001479 -17.51050923 17.631208420, -0.002939240

The calculated M is wrong (negative even).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.