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

Add color selection for photocal

    Details

    • Team:
      External

      Description

      Stars of certain colors (e.g., the upper branch in the gri diagram) are not suitable for use in measuring the photometric zeropoint. We know the color of stars in the reference catalog, so we can exclude them using that.

      This involves putting in a suitable Config entry to select the appropriate stars, and implement the selection within photocal.

        Attachments

          Issue Links

            Activity

            Hide
            price Paul Price added a comment -

            Thanks John.

            Merged to master.

            Show
            price Paul Price added a comment - Thanks John. Merged to master.
            Hide
            Parejkoj John Parejko added a comment -

            Hey, I just responded to a couple of your points, otherwise I think you're good.

            Show
            Parejkoj John Parejko added a comment - Hey, I just responded to a couple of your points, otherwise I think you're good.
            Hide
            price Paul Price added a comment -

            Jenkins is green, merging soon.

            Show
            price Paul Price added a comment - Jenkins is green, merging soon.
            Hide
            price Paul Price added a comment -

            Thanks John. I think you're done reviewing now and have only forgotten to switch the status here.

            I've tweaked meas_algorithms as well and am running everything through Jenkins. I would like to merge in the next day or so, so please shout soon if there are any objections to the current state.

            Show
            price Paul Price added a comment - Thanks John. I think you're done reviewing now and have only forgotten to switch the status here. I've tweaked meas_algorithms as well and am running everything through Jenkins. I would like to merge in the next day or so, so please shout soon if there are any objections to the current state.
            Hide
            Parejkoj John Parejko added a comment -

            Yes, I'm still finishing the meas_algorithms review.

            Show
            Parejkoj John Parejko added a comment - Yes, I'm still finishing the meas_algorithms review.

              People

              • Assignee:
                price Paul Price
                Reporter:
                price Paul Price
                Reviewers:
                John Parejko
                Watchers:
                John Parejko, Paul Price
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel