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

Integrate the M2 Python EUI with Common Module in Phase 1

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_main_telescope
    • Labels:

      Description

      Integrate the M2 Python EUI (ts_m2gui) with common module (ts_m2com) in phase 1. This is to reuse the controller in common module (which has been used in the M2 CSC, ts_m2). By doing this, the M2 Python EUI should be able to support the simulation mode and verify its functionality.

      The main point in this phase 1 is to remove the duplicated code in ts_m2gui and adapt the qasync.

        Attachments

          Issue Links

          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-35952 Integrate the M2 Python EUI with Common Module in Phase 2

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment - - edited

            I realized the pytest-qt and qasync can not be used at the same time for the event loop. The pytest-qt is just a wrapper of QtTest in PySide2. Therefore, it does not recognize the syntax of async or asyncSlot().

            One possible work-around is to write a simple function to update the existed code when doing the unit test.

            Show
            ttsai Te-Wei Tsai added a comment - - edited I realized the pytest-qt and qasync can not be used at the same time for the event loop. The pytest-qt is just a wrapper of QtTest in PySide2 . Therefore, it does not recognize the syntax of async or asyncSlot() . One possible work-around is to write a simple function to update the existed code when doing the unit test.
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            Created the issue in pytest-qt for this:
            https://github.com/pytest-dev/pytest-qt/issues/448

            Show
            ttsai Te-Wei Tsai added a comment - Created the issue in pytest-qt for this: https://github.com/pytest-dev/pytest-qt/issues/448
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            Please help to review the PR:
            https://github.com/lsst-ts/ts_m2gui/pull/17

            Thanks!

            Show
            ttsai Te-Wei Tsai added a comment - Please help to review the PR: https://github.com/lsst-ts/ts_m2gui/pull/17 Thanks!
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            Thanks Petr's review in GitHub!

            Show
            ttsai Te-Wei Tsai added a comment - Thanks Petr's review in GitHub!

              People

              Assignee:
              ttsai Te-Wei Tsai
              Reporter:
              ttsai Te-Wei Tsai
              Reviewers:
              Petr Kubanek
              Watchers:
              Petr Kubanek, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.