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

Import AUXTEL_LOCATION from obs_lsst instead of redefining (wrong)

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: atmospec, rapid_analysis
    • Labels:
      None
    • Story Points:
      0.25
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      Two packages have AUXTEL_LOCATION (re)defined (incorrectly). Import from obs_lsst where it's correct.

        Attachments

          Activity

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

          I wasn't expecting AUXTEL_LOCATION to be a part of the public API but I guess I walked into this one by not using a leading underscore...

          Show
          tjenness Tim Jenness added a comment - I wasn't expecting AUXTEL_LOCATION to be a part of the public API but I guess I walked into this one by not using a leading underscore...
          Hide
          Permalink
          mfisherlevine Merlin Fisher-Levine added a comment -

          I can copy and paste the instantiation rather than importing it if you prefer. That's what I was doing originally, I was just doing it wrong, and found the correct values there and thought I should import it instead of copying and pasting. Would that be better?

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - I can copy and paste the instantiation rather than importing it if you prefer. That's what I was doing originally, I was just doing it wrong, and found the correct values there and thought I should import it instead of copying and pasting. Would that be better?
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          if you need it it's obviously better to get it from obs_lsst. I was just thinking that I should have made it more explicitly public.

          Show
          tjenness Tim Jenness added a comment - if you need it it's obviously better to get it from obs_lsst. I was just thinking that I should have made it more explicitly public.
          Hide
          Permalink
          mfisherlevine Merlin Fisher-Levine added a comment -

          Yeah, it is a bit hidden down in the translators. This is why (in the package you may not have seen because it's not in lsst_distrib and Frossie is steadfastly ignoring my requests to get it added to Jira ) I import it and manually re-export it via _all_, so it's easily found.

          Show
          mfisherlevine Merlin Fisher-Levine added a comment - Yeah, it is a bit hidden down in the translators. This is why (in the package you may not have seen because it's not in lsst_distrib and Frossie is steadfastly ignoring my requests to get it added to Jira ) I import it and manually re-export it via _ all _, so it's easily found.

            People

            Assignee:
            mfisherlevine Merlin Fisher-Levine
            Reporter:
            mfisherlevine Merlin Fisher-Levine
            Reviewers:
            Christopher Waters
            Watchers:
            Christopher Waters, Merlin Fisher-Levine, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.