Skip to content
Snippets Groups Projects

Add calibration

Merged Colm Talbot requested to merge calibration into master
All threads resolved!

This MR adds calibration uncertainty to Interferometer objects.

Currently, just the cubic spline model used elsewhere is implemented.

An example of usage has been added.

  • Specifying the prior will be super boring so I should add a prior file for this.
Edited by Colm Talbot

Merge request reports

Pipeline #26933 passed

Pipeline passed for 122b48f5 on calibration

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

Merged by Colm TalbotColm Talbot 6 years ago (Aug 2, 2018 12:00am UTC)

Merge details

  • Changes merged into master with 6ebf16de.
  • Deleted the source branch.

Pipeline #27023 passed

Pipeline passed for 6ebf16de on master

Test coverage 100.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
  • Moritz Huebner
  • Moritz Huebner
  • Colm Talbot added 1 commit

    added 1 commit

    • fc2f1959 - fix logic on creating prior set

    Compare with previous version

  • Colm Talbot added 2 commits

    added 2 commits

    • 85579a24 - remove redundant statement
    • 62c2e473 - make instantiation with too few nodes less bizarre

    Compare with previous version

  • Colm Talbot added 2 commits

    added 2 commits

    • ba6cdac4 - set default calibration model explicitly
    • 30fafb46 - reomve redundant line

    Compare with previous version

  • Moritz Huebner approved this merge request

    approved this merge request

  • Colm Talbot added 7 commits

    added 7 commits

    Compare with previous version

  • Moritz Huebner resolved all discussions

    resolved all discussions

  • merged

  • Colm Talbot mentioned in commit 6ebf16de

    mentioned in commit 6ebf16de

  • Please register or sign in to reply
    Loading