Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DM-15528: Don't assume Camera iteration matches ccdExposureId ordering. #228

Merged
merged 1 commit into from
Oct 20, 2018
Merged

DM-15528: Don't assume Camera iteration matches ccdExposureId ordering. #228

merged 1 commit into from
Oct 20, 2018

Commits on Oct 20, 2018

  1. Don't assume Camera iteration matches ccdExposureId ordering. 

    The "observations" catalog used in the coadd tests needs to be
sorted for find() to work, but we never actually guaranteed this
in the past; the order was set by Camera iteration, which is
actually just dict iteration and hence has no guaranteed order.
This broke on DM-15528 when moving the Camera from Python
(dict) to C++ (unordered_map) changed that iteration order.
    @TallJimbo
    TallJimbo committed Oct 20, 2018
    Configuration menu
    Copy the full SHA
    5346256 View commit details
    Browse the repository at this point in the history