Skip to content

xx#241

Open
jack-sherman01 wants to merge 5 commits into
TideDra:mainfrom
jack-sherman01:main
Open

xx#241
jack-sherman01 wants to merge 5 commits into
TideDra:mainfrom
jack-sherman01:main

Conversation

@jack-sherman01

Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings April 23, 2026 23:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates runtime configuration to force a specific retrieval source when running the scheduled GitHub Actions workflow.

Changes:

  • Adjust config/custom.yaml executor settings (including commenting out prior example config).
  • Add a Hydra CLI override in the workflow to set executor.source at runtime.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
config/custom.yaml Modifies the configured executor source in the user override config.
.github/workflows/main.yml Adds a Hydra override to set executor.source when running the pipeline.

echo "Use custom config: "
cat config/custom.yaml
uv run src/zotero_arxiv_daily/main.py
uv run src/zotero_arxiv_daily/main.py executor.source=arxiv

Copilot AI Apr 23, 2026

Copy link

Choose a reason for hiding this comment

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

Hydra override executor.source=arxiv sets executor.source to a string, but the app expects it to be a list (it’s iterated over when building self.retrievers). Use a list override instead (e.g., executor.source=[arxiv] or executor.source=['arxiv']) to avoid iterating over individual characters.

Suggested change
uv run src/zotero_arxiv_daily/main.py executor.source=arxiv
uv run src/zotero_arxiv_daily/main.py executor.source=[arxiv]

Copilot uses AI. Check for mistakes.
Comment thread config/custom.yaml Outdated
executor:
debug: ${oc.env:DEBUG,null}
source: ['arxiv']
source:arxiv

Copilot AI Apr 23, 2026

Copy link

Choose a reason for hiding this comment

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

executor.source is expected to be a list (it’s iterated over in Executor.__init__), but this sets it to a scalar and also omits the required space after :. As written, it will either parse incorrectly or be treated as a string, causing iteration over characters ('a','r','x','i','v') and breaking retriever selection. Set executor.source to a proper YAML list (e.g., ['arxiv']).

Suggested change
source:arxiv
source: ['arxiv']

Copilot uses AI. Check for mistakes.
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.

3 participants