Skip to content

Examples for the accompanying blog post on search labs on how to use … #413

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

Merged
merged 2 commits into from
Mar 6, 2025

Conversation

peter-strsr
Copy link
Contributor

Code examples for Part 1 of the search labs article: https://github.com/elastic/search-labs-elastic-co/issues/1151

Copy link

gitnotebooks bot commented Mar 2, 2025

Found 1 changed notebook. Review the changes at https://app.gitnotebooks.com/elastic/elasticsearch-labs/pull/413

@JessicaGarson
Copy link
Contributor

JessicaGarson commented Mar 5, 2025

Hi, @peter-strsr,

It looks like the pre-commit checks are failing. You may want to try running the checks this way:

python -m venv .venv
.venv/bin/pip install -qqq -r requirements-dev.txt
.venv/bin/pre-commit install

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.

@peter-strsr
Copy link
Contributor Author

Thanks for the help @JessicaGarson and sorry for missing the contribution guide!

I formatted the notebooks and moved them to the other folder.

@JessicaGarson JessicaGarson merged commit afb9bcc into elastic:main Mar 6, 2025
2 checks passed
@peter-strsr peter-strsr deleted the colpali branch March 7, 2025 09:34
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