-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Remove community retrievers #8073
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 45 out of 45 changed files in this pull request and generated no comments.
abd74c9 to
1744621
Compare
|
Hi, @sam-h-bean @sky-2002 @AniLeo-01 @mogith-pn @insop @kingtroga @HARISHKUMAR1112001 @no-dice-io @PrashantDixit0 @jaelgu @tomasonjo @jhyearsley @poppingtonic @hasalams @drawal1 @andreswagner @sfc-gh-alherrera @ofermend. Thank you all for your great contributions to the DSPy retrievers. Due to the maintenance challenges and strategic shift, we will remove the community retrievers you added previously. If you use these retrievers in your system, please move the implementation into your repositories. Please let us know should you have any compelling reasons to retain them in the DSPy repository. Again, we deeply appreciate your support and look forward to your future contributions. Sincerely, DSPy maintenance team. |
f39b3d3 to
b4124db
Compare
b4124db to
6428351
Compare
|
@TomeHirata I understand the decision of removing the community retrievers as an architectural decision. However, is there a way to document them as examples in the DSPy documentation so the knowledge isn't lost? Alternatively, @tomasonjo, could I request as a "wish" that the Neo4jRM is documented in the neo4j documentation https://neo4j.com/labs/genai-ecosystem/dspy/ as a code snippet, without any commitment to update it to future DSPy versions; just as reference. Or am I missing something an retrievers are getting obsolete and being replaced with something else? |
|
@JPonsa We don't even know if the implementation is getting stale or still working, because we don't have maintainer with the domain knowledge of these vector index providers. Documentation is more robust if maintained by the retriever providers like neo4j than us, so I do agree with your call to add the documentation to neo4j site. |
Context
As part of the strategic shift of DSPy toward a reliable framework, we will stop supporting various community retrievers within the DSPy library since 2.7.0 due to the challenges of maintaining them reliably and the inactive status of their development.
Resolves #8072
Changes
Removed the following retrievers and related doc pages. Also, the retrieve dir is unified with retrievers dir.