Skip to content

Dev#207

Merged
pfahlr merged 11 commits into
mainfrom
dev
Sep 24, 2025
Merged

Dev#207
pfahlr merged 11 commits into
mainfrom
dev

Conversation

@pfahlr

@pfahlr pfahlr commented Sep 24, 2025

Copy link
Copy Markdown
Owner

No description provided.

@pfahlr pfahlr merged commit 319278f into main Sep 24, 2025
0 of 3 checks passed
pfahlr added a commit that referenced this pull request Sep 24, 2025
* Dev (#206)

* Fix verification harness index build commands

* Add index directory support to multi-agent RAG tooling

* Fix GPU dependency detection in CI

* Fix multi-agent invocation in verification harness

* fix(lc_ask): require key-derived faiss selection

* Dev (#207)

* Fix verification harness index build commands

* Add index directory support to multi-agent RAG tooling

* Fix GPU dependency detection in CI

* Fix multi-agent invocation in verification harness

* fix(lc_ask): require key-derived faiss selection

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment thread src/langchain/lc_ask.py
Comment on lines +353 to +357
if (cand / "index.faiss").exists():
faiss_dir = cand
break

if args.index_path:
if args.index_path:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[P0] Fix indentation in lc_ask main()

The new branch that handles --index vs --key is mis-indented. The line starting with if args.index_path: is dedented with three spaces relative to the four-space block above, so importing or running src/langchain/lc_ask.py now raises IndentationError: unindent does not match any outer indentation level before the CLI can even parse arguments. Try python -m py_compile src/langchain/lc_ask.py or python src/langchain/lc_ask.py to see the failure. The CLI is therefore unusable until the indentation is corrected.

Useful? React with 👍 / 👎.

pfahlr added a commit that referenced this pull request Sep 24, 2025
* Fix verification harness index build commands

* Add index directory support to multi-agent RAG tooling

* Fix GPU dependency detection in CI

* Fix multi-agent invocation in verification harness

* fix(lc_ask): require key-derived faiss selection

* mege (#208)

* Dev (#206)

* Fix verification harness index build commands

* Add index directory support to multi-agent RAG tooling

* Fix GPU dependency detection in CI

* Fix multi-agent invocation in verification harness

* fix(lc_ask): require key-derived faiss selection

* Dev (#207)

* Fix verification harness index build commands

* Add index directory support to multi-agent RAG tooling

* Fix GPU dependency detection in CI

* Fix multi-agent invocation in verification harness

* fix(lc_ask): require key-derived faiss selection
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.

1 participant