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-47531: refactor, test, and add ArrowAstropy support to ForcedPhotCcdTask. #292

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

DM-47531: refactor, test, and add ArrowAstropy support to ForcedPhotCcdTask. #292

wants to merge 5 commits into from
+503 −231

Commits on Nov 14, 2024

  1. Add unit test for ForcedPhotCcdTask.runQuantum.

    @TallJimbo
    TallJimbo committed Nov 14, 2024
    Configuration menu
    Copy the full SHA
    5f06b09 View commit details
    Browse the repository at this point in the history

  2. Add unit test for ForcedPhotCcdFromDataFrameTask.runQuantum.

    @TallJimbo
    TallJimbo committed Nov 14, 2024
    Configuration menu
    Copy the full SHA
    80d9300 View commit details
    Browse the repository at this point in the history

  3. Move DataFrame input support into base ForcedPhotCcdTask. 

    ForcedPhotCcd can now take DataFrame inputs by setting its
refCatStorageClass config parameter appropriately.  It also now
has config parameters for the columns it expects from the ref cat.

ForcedPhotCcdFromDataFrameTask still exists for backwards
compatibility, but now it's a trivial subclass that only sets some
config defaults.
    @TallJimbo
    TallJimbo committed Nov 14, 2024
    Configuration menu
    Copy the full SHA
    b21ddd1 View commit details
    Browse the repository at this point in the history

  4. Add ArrowAstropy input support to ForcedPhotCcd.

    @TallJimbo
    TallJimbo committed Nov 14, 2024
    Configuration menu
    Copy the full SHA
    5fc59cb View commit details
    Browse the repository at this point in the history

  5. Restore ForcedPhotCcdOnDataFrame connection defaults.

    @TallJimbo
    TallJimbo committed Nov 14, 2024
    Configuration menu
    Copy the full SHA
    e5e9e69 View commit details
    Browse the repository at this point in the history