Skip to content

feat(retrievers): Add option to set author, title etc. in ArxivRetriever #464

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.

Sign up for GitHub

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

Jump to bottom
Open
AIexanderDicke wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat(retrievers): Add option to set author, title etc. in ArxivRetriever #464

AIexanderDicke wants to merge 5 commits into from

Conversation

@AIexanderDicke
Copy link

@AIexanderDicke AIexanderDicke commented Dec 7, 2025

This PR

  • updates the ArXiv search functionality to use arxiv.Clientinstead of the deprecatedarxiv.Search`}
  • fixes the ArxivRetriever tests
  • fixes the file handling in lazy_load (also because the function from the arxiv lib will be deprecated)
  • adds the possibility to invoke the retriever with optional kwargs author, title, ... (see the details of query construction) to simplify the usage of the retriever.

@AIexanderDicke
fix:
652500f 
- _fetch_result uses arxiv.Client as arxiv.Search is deprecated
- uses min(self.load_max_docs, self.top_k_results) for max_results
- lazy_load no longer uses the pdf download from the arxiv-lib, also uses a temp dir
@AIexanderDicke
fix: tests need get_full_documents=True
44a84f7
@AIexanderDicke
feat: add optional kwargs to explicitly set title, author, ... for th…
cbd1437 
…e search
@AIexanderDicke
style: format & lint
7c47705
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@AIexanderDicke