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

photutils 1.8.0 breaks spectractor due to matplotlib usage

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: spectractor
    • Labels:
      None
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      This lsst_distrib clean build failed in the spectractor tests with this error from matplotlib called from photutils:

      ValueError: Can not reset the axes.  You are probably trying to re-use an artist in more than one Axes which is not supported
      

      The spectractor code has not changed, but photutils updated from 1.7.0 to 1.8.0. It seems that we may need to aggressively pin the photutils version in rubin-env.

        Attachments

          Activity

          Hide
          ktl Kian-Tat Lim added a comment -

          This turns out to be a change in the photutils.Background2D.plot_meshes() API. The use of axes was deprecated some time ago, but spectractor was never changed to the new API. Since this is not a photutils bug, I'm going to have to pin it, but this ticket should be used to fix spectractor.

          Show
          ktl Kian-Tat Lim added a comment - This turns out to be a change in the photutils.Background2D.plot_meshes() API. The use of axes was deprecated some time ago, but spectractor was never changed to the new API. Since this is not a photutils bug, I'm going to have to pin it, but this ticket should be used to fix spectractor.
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          On closer inspection, now that I'm back at the keyboard, I can see that this is indeed not a photutils problem, and is in fact already fixed in Spectractor upstream. I'm pulling in the changes elsewhere, so this will be fixed for free there, but I'll keep this ticket open until that's done, and then close this as well.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - On closer inspection, now that I'm back at the keyboard, I can see that this is indeed not a photutils problem, and is in fact already fixed in Spectractor upstream. I'm pulling in the changes elsewhere, so this will be fixed for free there, but I'll keep this ticket open until that's done, and then close this as well.
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          DM-38745 is where I'll resync with upstream to fix this.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - DM-38745 is where I'll resync with upstream to fix this.
          Hide
          erykoff Eli Rykoff added a comment -

          Merlin Fisher-Levine What's the status on this?

          Show
          erykoff Eli Rykoff added a comment - Merlin Fisher-Levine What's the status on this?
          Hide
          mfisherlevine Merlin Fisher-Levine added a comment -

          I believe this is now invalid, though I can't check the specific line as the failed build is back inside the old, dead Jenkins. Pretty sure this is behind us though, so if people agree, I think we close as Invalid now.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - I believe this is now invalid, though I can't check the specific line as the failed build is back inside the old, dead Jenkins. Pretty sure this is behind us though, so if people agree, I think we close as Invalid now.

            People

            Assignee:
            mfisherlevine Merlin Fisher-Levine
            Reporter:
            ktl Kian-Tat Lim
            Watchers:
            Eli Rykoff, Kian-Tat Lim, Merlin Fisher-Levine
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:

                Jenkins

                No builds found.