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

[NLP] Add support for the pass_through task #526

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

davidkyle
Copy link
Member

@davidkyle davidkyle commented Apr 6, 2023

The pass_through task returns the models output directly without any post-processing of the results. pass_through is typically used for debugging.

This change allows models to be loaded into Elasticsearch with the pass_through task

An example invocation of the eland_import_hub_model script with pass_through is:

eland_import_hub_model \
      --url XXX \
      -u XXX -p XXXX \
      --hub-model-id "sentence-transformers/msmarco-MiniLM-L-12-v3" \
      --task-type pass_through 

Copy link
Contributor

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@davidkyle davidkyle merged commit 940f2a9 into elastic:main Apr 6, 2023
@davidkyle davidkyle deleted the pass-through branch April 6, 2023 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants