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

Ensure TopKEncoder has correct outputs when model is saved #1225

Merged
merged 5 commits into from
Nov 10, 2023
Merged

Ensure TopKEncoder has correct outputs when model is saved #1225

merged 5 commits into from
Nov 10, 2023

Commits on Nov 8, 2023

  1. Remove output_names from base BaseModel.

    @oliverholworthy
    oliverholworthy committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    80a0f25 View commit details
    Browse the repository at this point in the history

  2. Add assertion for output signature of saved model to test_topk_encoder

    @oliverholworthy
    oliverholworthy committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    e02477f View commit details
    Browse the repository at this point in the history

  3. Move compile method from BaseModel to Model

    @oliverholworthy
    oliverholworthy committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    4c6af62 View commit details
    Browse the repository at this point in the history

  4. Correct name of structured outputs

    @oliverholworthy
    oliverholworthy committed Nov 8, 2023
    Configuration menu
    Copy the full SHA
    aed1b27 View commit details
    Browse the repository at this point in the history

Commits on Nov 9, 2023

  1. Move compile method back and add special case for TopKOutput

    @oliverholworthy
    oliverholworthy committed Nov 9, 2023
    Configuration menu
    Copy the full SHA
    7e14b13 View commit details
    Browse the repository at this point in the history