Skip to content

Set a default value for exposure_timeout parameter in JUNGFRAUConditions

Thomas Kluyver requested to merge test-ccapi2-jf into master

Description

The exposure_timeout parameter was added recently in !1066 (merged). It seems we have a logical default value of 25 which we ignore if specified, but we didn't put a default value in for JUNGFRAUCondition, so you have to pass it explicitly. When I synced this across to extra.calibration, a test which wasn't in this repo failed.

Is there a good reason to make it a required parameter? New required parameters mean breaking the API, whereas new optional ones don't.

How Has This Been Tested?

CI (added test in this MR)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Test (additional or refactored tests)

Checklist:

  • My code follows the code style of this project.

Reviewers

@ahmedk @schmidtp

Merge request reports