Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: ts_aos
-
Story Points:5
-
Team:Telescope and Site
-
Urgent?:No
Description
In ts_wep cwfs.Instrument gets the focus offset by converting the user input value into a string and then looking it up the value of the string in a separate configuration file. This needs to be more flexible for running at the summit and this lookup needs to be replaced by just using a value passed by the user.
Attachments
Issue Links
Activity
| Field | Original Value | New Value |
|---|---|---|
| Labels | UW |
| Status | To Do [ 10001 ] | In Progress [ 3 ] |
| Due Date | 31/Mar/22 |
| Labels | UW | AOS UW |
| Due Date | 31/Mar/22 | 30/Apr/22 |
| Remote Link | This issue links to "Page (Confluence)" [ 33037 ] |
| Story Points | 2 | 10 |
| Story Points | 10 | 15 |
| Story Points | 15 | 10 |
| Summary | Update Instrument class in ts_wep | Update Instrument class and tasks in ts_wep |
| Story Points | 10 | 5 |
Review of documentation showing how to use the new instrument configuration setup when running the Gen 3 AOS pipeline. On github here: https://github.com/lsst-ts/ts_analysis_notebooks/pull/28
| Reviewers | Tiago Ribeiro [ tribeiro ] | |
| Status | In Progress [ 3 ] | In Review [ 10004 ] |
Bruno Quint reviewed and approved documentation on the new changes on github.
| Status | In Review [ 10004 ] | Reviewed [ 10101 ] |
| Resolution | Done [ 10000 ] | |
| Status | Reviewed [ 10101 ] | Done [ 10002 ] |
The unit test on Jenkins stuck on the Jenkins build. Rob Bovill kindly suggested to add the following label in in the block of agent in Jenkinsfile:
label 'Node2_8CPU'Therefore,
agent {// Use the docker to assign the Python version.// Use the label to assign the node to run the test.// It is recommended by SQUARE team do not add the label to let the// system decide.// But in some cases, we do need to put the label to assign the specific// node to use.docker {image 'lsstts/develop-env:develop'args "-u root --entrypoint=''"label 'Node2_8CPU'}}Please consider this. Thanks!