obs_lsst failing LATISS plate scale test

Description

In DM-23490 the plate scale of LATISS was changed but the tests were not run before merging. This is breaking master:

self = <test_generateCamera.PhosimToRaftsTestCase testMethod=testGenerateCameraLatiss>
    def testGenerateCameraLatiss(self):
        """Test with LATISS in a test directory."""
        content = self.runGenerateCamera(["latiss", "lsstCam", os.path.curdir])
        self.assertEqual(content["name"], "LATISS")
>       self.assertEqual(content["plateScale"], 20.0)
E       AssertionError: 10.112 != 20.0

I will fix the plate scale in the tests.

Linked work items

is triggered by

DM-23490

Correct plate scale and boresight position for LATISS

Checklist

Lucidchart Diagrams

Issue Matrix

hide

Activity

Show:

John Parejko

February 14, 2020 at 2:01 AM

(edited)

"The tests were run on the summit machines, but this failed to show up." - meaning the tests did in fact, not pass.

Merlin Fisher-Levine

February 14, 2020 at 1:55 AM

Go ahead and merge.

Tim Jenness

February 14, 2020 at 1:54 AM

putting in review with you but will merge since you reviewed and tested on slack for me.

Robert Lupton

February 14, 2020 at 1:49 AM

The tests were run on the summit machines, but this failed to show up. It may have been masked by other problems related to finding the home directory.

Resize issue view side panel

Done

Pinned fields

Click on the next to a field label to start pinning.

Details

Assignee

Tim Jenness

Reporter

Tim Jenness

Reviewers

Merlin Fisher-Levine

Story Points

0.5

RubinTeam

Architecture

Components

obs_lsst

Checklist

Created February 14, 2020 at 1:20 AM

Updated February 14, 2020 at 4:26 AM

Resolved February 14, 2020 at 4:26 AM