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.

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

DM-34175: Simplify ingest with extended exposure table and related dimensions #54

Merged
merged 1 commit into from
Apr 5, 2022

Commits on Apr 5, 2022

  1. allow extensions of the core set of properties

    Some instruments have important properties beyond the core set
    that have been defined, and we want to be able to extract those
    as well. Added the concept of 'extensions' to both the
    MetadataTranslator and ObservationInfo. This allows instruments
    to define additional instrument-specific properties in their
    appropriate MetadataTranslator, and these get picked up by the
    ObservationInfo.
    
    Unfortunately, extension properties have to get defined in the
    ObservationInfo at runtime (because we don't know what they are
    at compile time), which means we need to use a different scheme
    for making the fields read-only, and there's no clear place to
    hang their docstrings. But this is counteracted by the fact that
    users don't need to choose a specific class to use, or define
    the extensions themselves.
    PaulPrice committed Apr 5, 2022
    Configuration menu
    Copy the full SHA
    2b33bdd View commit details
    Browse the repository at this point in the history