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

Apparent issue in efd_client when unpacking packed arrays

    XMLWordPrintable

    Details

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

      Description

      We see time drifts between packed and unpacked data that has been attributed to issues inside the cRIO. However, we think there is also a bug in the efd_client.

      Attached is a notebook detailing the issue. Essentially the index times after unpacking do not match what is expected.

        Attachments

        1. Packed_timing_issue.ipynb
          162 kB
        2. Packed_timing_issue.pdf
          68 kB
        3. TAI_Check_04May22.png
          TAI_Check_04May22.png
          201 kB
        4. TAI_Difference_09Mar22.png
          TAI_Difference_09Mar22.png
          113 kB
        5. Time_Discrepancy_08Mar22.png
          Time_Discrepancy_08Mar22.png
          409 kB

          Issue Links

          relates to

          Task - A task that needs to be done. CAP-816 High-Frequency Data Array Timestamp Definition

          • Blocker - Blocks development and/or testing work, production could not run.
          • To Do

            Activity

            Ascending order - Click to sort in descending order
            5 older comments
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            https://github.com/astropy/astropy/issues/11245

            5.0 has the fix.

            Show
            tjenness Tim Jenness added a comment - https://github.com/astropy/astropy/issues/11245 5.0 has the fix.
            Hide
            Permalink
            cslage Craig Lage added a comment -

            I saw that ticket, but I didn't find where 5.0 has the fix. What does it take to update the summit?

            Show
            cslage Craig Lage added a comment - I saw that ticket, but I didn't find where 5.0 has the fix. What does it take to update the summit?
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            I think I saw Andy Clements and Tiago Ribeiro talking about the bad astropy version on Slack a while back.

            Show
            tjenness Tim Jenness added a comment - I think I saw Andy Clements and Tiago Ribeiro talking about the bad astropy version on Slack a while back.
            Hide
            Permalink
            cslage Craig Lage added a comment -

            Tiago says, "yes.. we are set to upgrade soon.. there was a dependency problem we solved recently". After the upgrade I'll re-test to make sure that fixes the problem.

            Show
            cslage Craig Lage added a comment - Tiago says, "yes.. we are set to upgrade soon.. there was a dependency problem we solved recently". After the upgrade I'll re-test to make sure that fixes the problem.
            Hide
            Permalink
            cslage Craig Lage added a comment -

            I confirmed that the 8 second discrepancy between the summit and NCSA is now fixed. See the attached TAI_Check_04May22.png

            Show
            cslage Craig Lage added a comment - I confirmed that the 8 second discrepancy between the summit and NCSA is now fixed. See the attached TAI_Check_04May22.png

              People

              Assignee:
              frossie Frossie Economou
              Reporter:
              pingraham Patrick Ingraham
              Watchers:
              Craig Lage, Patrick Ingraham, Tiago Ribeiro, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.