Skip to content

Coaster Searching Prep: basic document embedding & searching w/ ollama bindings - #13

Merged
jackjustus merged 22 commits into
mainfrom
coaster-search
Aug 15, 2026
Merged

jackjustus merged 22 commits into
mainfrom
coaster-search

Conversation

@jackjustus

@jackjustus jackjustus commented Aug 14, 2026 •

Copy link
Copy Markdown
Owner

@zachxwalton this pr implements the search endpoint by:

  1. backfilling embeddings (via ollama's nomic-embed-text) for all coasters in the db using the metadata we have.
  2. embedding text queries which hit the search endpoint and finding the nearest neighbor using cosine distance.

this allows us to do cool things that keyword /trigram searching wouldn't like "that one ride in utah named rollercoaster" and we still get Roller Coaster @ Lagoon.

Follow up PRs:

  • CLI Update to use the new search endpoint
  • Improving the richness of our db's metadata on coasters to embed more info, so we can effectively respond to queries such as "the tallest coaster in ohio" or "that one defunct ride at six flags new jersey".
  • a benchmark to compare different document structure's effectiveness at common queries.

happy to discuss anything.

@jackjustus jackjustus linked an issue Aug 14, 2026 that may be closed by this pull request
@jackjustus jackjustus removed a link to an issue Aug 14, 2026
@jackjustus jackjustus linked an issue Aug 14, 2026 that may be closed by this pull request
@jackjustus jackjustus changed the title coaster-search Coaster Searching Prep: db migrations & ollama bindings Aug 14, 2026
@jackjustus jackjustus changed the title Coaster Searching Prep: db migrations & ollama bindings Coaster Searching Prep: basic document embedding & searching w/ ollama bindings Aug 15, 2026
@jackjustus
jackjustus requested review from zachxwalton and a balanced review from Copilot August 15, 2026 00:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

zachxwalton
zachxwalton previously approved these changes Aug 15, 2026

@zachxwalton zachxwalton left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm king

# Conflicts:
#	.github/workflows/ci.yml
#	Makefile
@jackjustus

Copy link
Copy Markdown
Owner Author

thanks king

@jackjustus
jackjustus merged commit 21faace into main Aug 15, 2026
9 checks passed
@jackjustus
jackjustus deleted the coaster-search branch August 15, 2026 01: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.

Search initial implementation - embedding basic data about coasters.

3 participants