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

Update lsst_sims to use obs_lsst instead of obs_lsstSim

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_lsst, obs_lsstSim
    • Labels:
      None

      Description

      Some code in lsst_sims uses the camera description to translate between sky coordinates and pixel coordinates. This is blocked on implementing RFC-571.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Why can't we merge those now? If they work there is no need to wait for obs_lsstSim to disappear.

            Show
            tjenness Tim Jenness added a comment - Why can't we merge those now? If they work there is no need to wait for obs_lsstSim to disappear.
            Hide
            swinbank John Swinbank added a comment -

            From the PR messages:

            Do not merge it beforehand, as it will delete (as opposed to replacing) functionality that currently depends on obs_lsstSim

            I understand the Sims team are ok with that, but don't want to lose that functionality before they have to.

            Show
            swinbank John Swinbank added a comment - From the PR messages: Do not merge it beforehand, as it will delete (as opposed to replacing) functionality that currently depends on obs_lsstSim I understand the Sims team are ok with that, but don't want to lose that functionality before they have to.
            Hide
            tjenness Tim Jenness added a comment -

            Do we know what's going on here? These PRs still seem to be open and we removed obs_lsstSim a while back. Should we merge them? Why has no-one noticed they haven't merged?

            Show
            tjenness Tim Jenness added a comment - Do we know what's going on here? These PRs still seem to be open and we removed obs_lsstSim a while back. Should we merge them? Why has no-one noticed they haven't merged?
            Hide
            tjenness Tim Jenness added a comment -

            Lynne Jones can I assume this work was done a long time ago and we can close this ticket?

            I see that the sims_GalSimInterface PR is still open, the sims_catUtils one has been merged, the sims_coordUtils one has been closed without merging and all 3 packages are now deprecated.

            Show
            tjenness Tim Jenness added a comment - Lynne Jones can I assume this work was done a long time ago and we can close this ticket? I see that the sims_GalSimInterface PR is still open, the sims_catUtils one has been merged, the sims_coordUtils one has been closed without merging and all 3 packages are now deprecated.
            Hide
            ljones Lynne Jones added a comment -

            Yes, all 3 packages are deprecated. I'm guessing we didn't merge them because we didn't need to for some reason? However, we no longer have a dependency on obs_lsst or obs_lsstSim for rubin_sim. 

            Show
            ljones Lynne Jones added a comment - Yes, all 3 packages are deprecated. I'm guessing we didn't merge them because we didn't need to for some reason? However, we no longer have a dependency on obs_lsst or obs_lsstSim for rubin_sim. 

              People

              Assignee:
              danielsf Scott Daniel
              Reporter:
              krughoff Simon Krughoff
              Watchers:
              Lynne Jones, Scott Daniel, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.