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-27153: adapt to daf_butler changes to defaulting of collections and data IDs #110

Merged
merged 3 commits into from
Jan 12, 2021
Merged

DM-27153: adapt to daf_butler changes to defaulting of collections and data IDs #110

merged 3 commits into from
Jan 12, 2021

Commits on Jan 11, 2021

  1. Improve typing and docs for collection sequences. 

    DatasetTypeRestriction hasn't been used here for a while, so those
annotations and docs are just wrong.  Docs about the inputs attribute
being updated to include outputs were wrong, too, and because it isn't
being updated, I've made it a tuple instead of a list so there's no
doubt about that.
    @TallJimbo
    TallJimbo committed Jan 11, 2021
    Configuration menu
    Copy the full SHA
    47e08f1 View commit details
    Browse the repository at this point in the history

  2. Move collection creation out of butler constructor call. 

    This is necessary because Butler.__init__ no longer has chains argument
that creates CHAINED collections, but this makes what's going on here
more explicit and avoids some necessary butler operations (no-ops, but
the kind that still involve database calls) by making better use of our
knowledge of command-line options.
    @TallJimbo
    TallJimbo committed Jan 11, 2021
    Configuration menu
    Copy the full SHA
    0cf2aab View commit details
    Browse the repository at this point in the history

  3. Use public Registry.refresh() now that it exists.

    @TallJimbo
    TallJimbo committed Jan 11, 2021
    Configuration menu
    Copy the full SHA
    a0f2fe8 View commit details
    Browse the repository at this point in the history