sync: bring rag-params-finder CodeRabbit fixes onto fork main - #5
Merged
Merged
Conversation
Export MONGODB_URI before the SIE handoff, recreate Compose for env reload, require encode readiness before sweeps, and document the gitignored input PDF prerequisite for example-sie.yaml. Co-authored-by: Cursor <cursoragent@cursor.com>
Avoid sourcing .env wholesale (it clobbers the host CLI Atlas Local URI) and stop the encode readiness loop on terminal HTTP failures. Co-authored-by: Cursor <cursoragent@cursor.com>
Port CodeRabbit patterns from superlinked#244: separate local vs optional SIE paths, encode/score primitive naming, config-specific Mongo indexes, and an external-guide submission path in the gallery README. Co-authored-by: Cursor <cursoragent@cursor.com>
Align the readiness-loop comment with the real ~40m worst-case budget, and state that self-hosted SIE requires HF_TOKEN for model downloads. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
neomatrix369/siemainwith the tip ofdocs/upstream-rag-params-finderused by superlinked/sie#241Test plan
examples/rag-params-finder/**+ gallery README updatesmainMade with Cursor