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

MTMount OSS telemetry topic causes Java too many parameters error

    XMLWordPrintable

Details

    • 2
    • TSSW Sprint - Nov 25 - Dec 6, TSSW Sprint - Dec 9 - Dec 20, TSSW Sprint - Dec 23 - Jan 3, TSSW Sprint - Jan 06 - Jan 17, TSSW Sprint - Jan 21 - Feb 01
    • Telescope and Site

    Description

      This is a ticket to track the Java "too many parameters" error in the MtMount OSS telemetry topic.  This issue will probably go away when the vendor updates the topics.  QA needs the ticket to properly shutdown the tests that are affected.

      Attachments

        Activity

          rbovill Rob Bovill added a comment -

          This is still an issue, so I'm making this comment to ensure this isn't forgotten.

          rbovill Rob Bovill added a comment - This is still an issue, so I'm making this comment to ensure this isn't forgotten.

          Thanks Rob, I will look into this within the next few days. 

          aheyer Andrew Heyer [X] (Inactive) added a comment - Thanks Rob, I will look into this within the next few days. 

          Hi Rob, PR created here https://github.com/lsst-ts/ts_xml/pull/22

          It seems that the only test that wasn't passing was a unit field Amp. I also rebased and removed MTMount commands being skipped. 

          I recall there being many more issues when I initially looked at it, now I think I was mistaken?

          aheyer Andrew Heyer [X] (Inactive) added a comment - Hi Rob, PR created here https://github.com/lsst-ts/ts_xml/pull/22 It seems that the only test that wasn't passing was a unit field Amp. I also rebased and removed MTMount commands being skipped.  I recall there being many more issues when I initially looked at it, now I think I was mistaken?
          rbovill Rob Bovill added a comment -

          Please fix the <Unit> fields in the Telemetry XML. It would also be good to update ts_xml/tests/test_Units.py to catch this condition.

          rbovill Rob Bovill added a comment - Please fix the <Unit> fields in the Telemetry XML. It would also be good to update ts_xml/tests/test_Units.py to catch this condition.

          added a test to catch for blank spaces and unit attributes have been fixed.

          aheyer Andrew Heyer [X] (Inactive) added a comment - added a test to catch for blank spaces and unit attributes have been fixed.
          rbovill Rob Bovill added a comment -

          MTMount XML changes approved. Tests now passing. Updates to Unit tests are also good. See https://github.com/lsst-ts/ts_xml/pull/224 for details.

          rbovill Rob Bovill added a comment - MTMount XML changes approved. Tests now passing. Updates to Unit tests are also good. See https://github.com/lsst-ts/ts_xml/pull/224 for details.

          People

            aheyer Andrew Heyer [X] (Inactive)
            rbovill Rob Bovill
            Rob Bovill
            Andrew Heyer [X] (Inactive), Rob Bovill
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.