Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-18676

Finish first release for electrometer

    Details

      Description

       
      Clean up Electrometer python code and add limits when applies (for example integration time should be limited). Also create a docker image for deployment.

      Update: 

        Attachments

          Issue Links

            Activity

            Hide
            aanania Andres Anania [X] (Inactive) added a comment - - edited

            There is a release for the electrometer that uses my own implementation of configuration. Things TBD:

            • It needs to be updated to the implementation in salobj. 
            • Also to new salobj version

            Tasks related:

            https://jira.lsstcorp.org/browse/DM-18674

            https://jira.lsstcorp.org/browse/DM-18590

            Show
            aanania Andres Anania [X] (Inactive) added a comment - - edited There is a release for the electrometer that uses my own implementation of configuration. Things TBD: It needs to be updated to the implementation in salobj.  Also to new salobj version Tasks related: https://jira.lsstcorp.org/browse/DM-18674 https://jira.lsstcorp.org/browse/DM-18590
            Hide
            aclements Andy Clements added a comment -

            need to review

            Show
            aclements Andy Clements added a comment - need to review
            Hide
            aclements Andy Clements added a comment -

            Talked to Patrick.  He has said that Andres has updated the code to current coding standards.  Eric Coughlin has said he has worked with it and see no problems.  Story reviewed.  

            Show
            aclements Andy Clements added a comment - Talked to Patrick.  He has said that Andres has updated the code to current coding standards.  Eric Coughlin has said he has worked with it and see no problems.  Story reviewed.  

              People

              • Assignee:
                aanania Andres Anania [X] (Inactive)
                Reporter:
                aanania Andres Anania [X] (Inactive)
                Reviewers:
                Andy Clements
                Watchers:
                Andres Anania [X] (Inactive), Andy Clements, Eric Coughlin, Patrick Ingraham
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel