-
Notifications
You must be signed in to change notification settings - Fork 224
deepeval-notebook #403
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
deepeval-notebook #403
Conversation
Found 1 changed notebook. Review the changes at https://app.gitnotebooks.com/elastic/elasticsearch-labs/pull/403 |
💚 CLA has been signed |
@kritinv please sign the CLA (https://www.elastic.co/contributor-agreement) with the email you used in Git / GitHub for making the PR. |
Hey @xeraa, I believe the CLA has been signed. Thanks! |
Great, the CLA is done (and the comment updated — I was briefly confused by the timeline). Since I'm also traveling this week, @JessicaGarson can you please take a look at the failures of the PR (might be something more cosmetic) and merge it. Then we can also get the blog post published. |
Thanks, @kritinv. Sine the pre-commit checks are failing. You may want to try running the checks this way:
You can find more infomation in our contributing guide. If you have issues after that, try moving the notebook to the supporting-blog-content folder, since this notebook corresponds to a blog post. This folder is more flexible when it comes to formatting checks. |
I think this has been fixed! @JessicaGarson |
Hi @kritinv, It still looks like the checks are failing here - the easiest way to resolve this is moving this post from |
@JessicaGarson moved it! |
This notebook is a tutorial on evaluating and picking the best hyperparameters for a RAG system using an Elasticsearch retriever, with DeepEval. It follows the contents of the corresponding blog.