Skip to content
Snippets Groups Projects

OT t0 time tools provoke FP auditor into crashing

Merged Manuel Tobias Schiller requested to merge mschille-20190506-ott0-nanfix into run2-patches

Gaudi::Math::ValueWithError's constructor and error methods are not transparent to quiet NaNs, since they do arithmetic on the constructor's arguments. Until we've figured out a good way to deal with that that impacts nobody else, we change the type returned by the OT track t0 tools (with a type alias) to something much simpler that is transparent to NaNs. Fix one place where a NaN produced by the OT track t0 tools confused the packing, while we're at it.

Edited by Manuel Tobias Schiller

Merge request reports

Approval is optional

Merged by Eduardo RodriguesEduardo Rodrigues 5 years ago (May 7, 2019 7:33am UTC)

Merge details

  • Changes merged into run2-patches with 7ebe4d2f.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading