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

Write how to make gen3 obs package guide

    Details

      Description

      While doing the gen3 DECam work, I started a document describing how to write a gen3 obs package. That document will be fleshed out as part of doing the obs_lsst gen3 cleanup, and this ticket is to finally publish it. It will live in obs_base/docs/lsst.obs.base/.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            This is on hold while we sort out remaining design issues.

            Show
            swinbank John Swinbank added a comment - This is on hold while we sort out remaining design issues.
            Hide
            Parejkoj John Parejko added a comment -

            Tim Jenness: is the gen3 obs system stable enough that this could finally be completed? I suspect that I am no longer in a good place to write it now, given changes since I last worked on the obs packages. I've seen the queries from various people wanting to write gen3 obs or update their gen2 obs package: might be nice to take care of all of those questions at once with these docs.

            Show
            Parejkoj John Parejko added a comment - Tim Jenness : is the gen3 obs system stable enough that this could finally be completed? I suspect that I am no longer in a good place to write it now, given changes since I last worked on the obs packages. I've seen the queries from various people wanting to write gen3 obs or update their gen2 obs package: might be nice to take care of all of those questions at once with these docs.
            Hide
            j.mullaney James Mullaney added a comment -

            For what it's worth, I've just finished making obs_necam gen3 compliant....
            https://github.com/jrmullaney/obs_necam

            Show
            j.mullaney James Mullaney added a comment - For what it's worth, I've just finished making obs_necam gen3 compliant.... https://github.com/jrmullaney/obs_necam
            Hide
            tjenness Tim Jenness added a comment -

            James Mullaney excellent. Note that you don't need to call it lsst.obs...

            Let me know if you want some feedback.

            John Parejko, yes, it's on the list. I'll assign it to me.

            Show
            tjenness Tim Jenness added a comment - James Mullaney excellent. Note that you don't need to call it lsst.obs... Let me know if you want some feedback. John Parejko , yes, it's on the list. I'll assign it to me.
            Hide
            j.mullaney James Mullaney added a comment -

            @tjenness Yes, feedback would be great, thanks. Perhaps best to put them as Issues in the GitHub repository. Be kind
            Not sure what you meant by "you don't need to call it lsst.obs...", though. Do you mean in the bin/run_gen3_demo.sh file?

            Show
            j.mullaney James Mullaney added a comment - @tjenness Yes, feedback would be great, thanks. Perhaps best to put them as Issues in the GitHub repository. Be kind Not sure what you meant by "you don't need to call it lsst.obs...", though. Do you mean in the bin/run_gen3_demo.sh file?
            Hide
            tjenness Tim Jenness added a comment -

            I mean in your obs_necam package itself. Gen3 doesn't require all obs packages use lsst.obs. in the module name. Obviously you can do that but you don't need to.

            Show
            tjenness Tim Jenness added a comment - I mean in your obs_necam package itself. Gen3 doesn't require all obs packages use lsst.obs. in the module name. Obviously you can do that but you don't need to.
            Hide
            j.mullaney James Mullaney added a comment -

            Oh, I see what you mean. Thanks for explaining that.

            Show
            j.mullaney James Mullaney added a comment - Oh, I see what you mean. Thanks for explaining that.
            Hide
            tjenness Tim Jenness added a comment -

            John Parejko I updated your text a little to keep it up to date but decided to merge now so that it's in the next weekly. I reassigned it to you since you did the work and I reviewed.

            Show
            tjenness Tim Jenness added a comment - John Parejko I updated your text a little to keep it up to date but decided to merge now so that it's in the next weekly. I reassigned it to you since you did the work and I reviewed.

              People

              • Assignee:
                Parejkoj John Parejko
                Reporter:
                Parejkoj John Parejko
                Watchers:
                James Mullaney, Jim Bosch, John Parejko, John Swinbank, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: