-
Notifications
You must be signed in to change notification settings - Fork 69
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
More documentation please #49
Comments
@jusjosgra where you able to train custom model and use it with nboost. I'll be interested to know how you did it. |
@jusjosgra Do you have the required data for benchmarking?? |
@jusjosgra Thank you for your positive feedback - I will try to improve the documentation. Essentially the way it works is that there are a set of config variables (which can be viewed by running As for these two specific questions: Nboost expects a binary classifier model trained as 0 = not relevant to search 1 = relevant to search. |
Thanks so much for your thorough response! For no_rerank, it appears that this is specified at proxy deploy time, is there a way to pass it at query time to skip the reranking step dynamically? |
@jusjosgra all of these flags are both deploy time and dynamic, as you can include it in the json body or query params of the request to override. For example if request is a POST with the following JSON body: {
"query" : "This is a search",
"nboost" : {
"no_rerank" : true
}
} or if it is a GET request simply append it to the url http://nboost_host:8000?q=query&no_rerank=True |
Hey @pertschuk, Yes I was able to that but it still returns the same results both for
|
This library is excellent and I would really like to use it, so first of all thank you for making it available!
Unfortunately, there isnt enough documentation to make it particularly usable beyond a default deployment. For example it is not clear how I could:
I am digging through the codebase to solve these issues but it would be great if you could document the API as I am sure these things are straight forward.
Thanks again for the great library.
The text was updated successfully, but these errors were encountered: