You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There seems a bug in FullLDCTrackingAlg when I disable dumping the ntuples (dumpTime=False):
- In line 246 and 582 if(m_tuple), which I think it's better to write as if(m_dumpTime && m_tuple) to avoid the break.
This is such a minor debug so I did not create a PR, maybe you can fix it sometime convenient.
Hi,
There seems a bug in FullLDCTrackingAlg when I disable dumping the ntuples (dumpTime=False):
- In line 246 and 582
if(m_tuple)
, which I think it's better to write asif(m_dumpTime && m_tuple)
to avoid the break.This is such a minor debug so I did not create a PR, maybe you can fix it sometime convenient.
Cheers,
Fangyi
*Code in git:
https://github.com/cepc/CEPCSW/blob/master/Reconstruction/Tracking/src/FullLDCTracking/FullLDCTrackingAlg.cpp#L246
https://github.com/cepc/CEPCSW/blob/master/Reconstruction/Tracking/src/FullLDCTracking/FullLDCTrackingAlg.cpp#L582
The text was updated successfully, but these errors were encountered: