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

Implement interface for bulk inferencing in TF models #8560

Merged
merged 10 commits into from
Apr 28, 2021
Merged

Implement interface for bulk inferencing in TF models #8560

merged 10 commits into from
Apr 28, 2021

Commits on Apr 27, 2021

  1. use data generator for inference

    @dakshvar22
    dakshvar22 committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    e2705c3 View commit details
    Browse the repository at this point in the history

  2. refactor to create the generator inside to avoid duplication

    @dakshvar22
    dakshvar22 committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    6fd3fff View commit details
    Browse the repository at this point in the history

  3. added tests and changelog

    @dakshvar22
    dakshvar22 committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    ac764ec View commit details
    Browse the repository at this point in the history

Commits on Apr 28, 2021

  1. Apply suggestions from code review 

    Co-authored-by: Sam Sucik <[email protected]>
    @dakshvar22 @samsucik
    dakshvar22 and samsucik authored Apr 28, 2021
    Configuration menu
    Copy the full SHA
    9eb8a2b View commit details
    Browse the repository at this point in the history

  2. review comments

    @dakshvar22
    dakshvar22 committed Apr 28, 2021
    Configuration menu
    Copy the full SHA
    0c5ae99 View commit details
    Browse the repository at this point in the history

  3. fix docstring

    @dakshvar22
    dakshvar22 committed Apr 28, 2021
    Configuration menu
    Copy the full SHA
    e08e649 View commit details
    Browse the repository at this point in the history

  4. place merge method inside RasaModel. Make rasa_predict private

    @dakshvar22
    dakshvar22 committed Apr 28, 2021
    Configuration menu
    Copy the full SHA
    0517008 View commit details
    Browse the repository at this point in the history

  5. satisfy linter

    @dakshvar22
    dakshvar22 committed Apr 28, 2021
    Configuration menu
    Copy the full SHA
    a092ba7 View commit details
    Browse the repository at this point in the history

  6. Update rasa/utils/tensorflow/models.py 

    Co-authored-by: Sam Sucik <[email protected]>
    @dakshvar22 @samsucik
    dakshvar22 and samsucik authored Apr 28, 2021
    Configuration menu
    Copy the full SHA
    0bd1f5c View commit details
    Browse the repository at this point in the history

  7. change annotation type

    @dakshvar22
    dakshvar22 committed Apr 28, 2021
    Configuration menu
    Copy the full SHA
    82f90e5 View commit details
    Browse the repository at this point in the history