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-1331 #5

Merged
merged 4 commits into from
Oct 13, 2014
Merged

DM-1331 #5

merged 4 commits into from
Oct 13, 2014

Commits on Oct 10, 2014

  1. Swap flag definition order to match enum order

    @TallJimbo
    TallJimbo committed Oct 10, 2014
    2 Configuration menu
    Copy the full SHA
    e70e080 View commit details
    Browse the repository at this point in the history

Commits on Oct 13, 2014

  1. Refactor SdssCentroid test to better use test utilities 

    The first several lines of the previous version could be accomplished
by simply inheriting from meas.base.tests.AlgorithmTestCase, which
additionally puts that code in setUp(), allowing it to be reused
for other test functions.

Also fixed non-conformant variable names.
    @TallJimbo
    TallJimbo committed Oct 13, 2014
    Configuration menu
    Copy the full SHA
    40a197b View commit details
    Browse the repository at this point in the history

  2. Catch exception in SdssCentroid to replace log warning with set flag

    @TallJimbo
    TallJimbo committed Oct 13, 2014
    Configuration menu
    Copy the full SHA
    3307cc4 View commit details
    Browse the repository at this point in the history

  3. Add edge flag for NaiveFlux, to account for cases when aperture doesn… 

    …'t fit in frame
    @TallJimbo
    TallJimbo committed Oct 13, 2014
    Configuration menu
    Copy the full SHA
    6cae377 View commit details
    Browse the repository at this point in the history