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

Explicitly Type CorpusSample in ES #369

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

Explicitly Type CorpusSample in ES #369

wants to merge 2 commits into from

Commits on Sep 16, 2024

  1. Explicitly Type CorpusSample in ES 

    This patch makes blackbox_learner pass a CorpusSample object to compile
jobs rather than a list of ModuleSpecs, making things a bit more
readable as now we hold a list of CorpusSamples rather than nested lists
of ModuleSpecs.
    @boomanaiden154
    boomanaiden154 committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    ed2db38 View commit details
    Browse the repository at this point in the history

  2. Linting

    @boomanaiden154
    boomanaiden154 committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    2adb07d View commit details
    Browse the repository at this point in the history