Skip to content
Snippets Groups Projects

Fixing bug in dt

Merged Marc Arene requested to merge timeshift into master

Computing dt the following way is inaccurate:

dt = parameters['geocent_time'] + time_shift - self.strain_data.start_time

Indeed the GPS times are number on the order of 1e9 seconds while timeshift varies on the order of 1e-5. Hence if the GPS times are not substracted with one another before the computation is inaccurate.

Merge request reports

Pipeline #78037 passed

Pipeline passed for f4d06264 on timeshift

Test coverage 70.00% (0.00%) from 1 job
Approved by

Merged by Gregory AshtonGregory Ashton 5 years ago (Sep 3, 2019 2:05pm UTC)

Merge details

  • Changes merged into master with 25539b62 (commits were squashed).
  • Deleted the source branch.

Pipeline #78190 passed with warnings

Pipeline passed with warnings for 25539b62 on master

Test coverage 70.00% (0.00%) from 1 job

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