Skip to content
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

AIMonRerank.ipynb notebook added to LlamaIndex docs/examples #18088

Draft
wants to merge 29 commits into
base: main
Choose a base branch
from
Draft

AIMonRerank.ipynb notebook added to LlamaIndex docs/examples #18088

wants to merge 29 commits into from

Conversation

devvratbhardwaj
Copy link
Contributor

Description

Added the AIMon reranker notebook to the LlamaIndex documentation. This PR is stacked on top of PR #18087. Please review that first.

Type of Change

  • This change requires a documentation update

How Has This Been Tested?

Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

  • I believe this change is already covered by existing unit tests

Suggested Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added Google Colab support for the newly added notebooks.
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I ran make format; make lint to appease the lint gods

devvratbhardwaj and others added 28 commits March 4, 2025 18:26
@devvratbhardwaj
aimon-reranker-integration
9917b9a
@devvratbhardwaj
Declaring task_definition as a Field
f5ac629
@devvratbhardwaj
updating the arguments for aimon-reranker call
da63c0a
@devvratbhardwaj
modified post_processing according to aimon reranking results
b097195
@devvratbhardwaj
top_n filtering added
5fc7089
@devvratbhardwaj
update to post processing
a1d67dd
@devvratbhardwaj
update+=2
289a02f
@devvratbhardwaj
POST request approach
b171cb2
@devvratbhardwaj
printing response_data received from AIMon
b408ba4
@devvratbhardwaj
update+=3
5d4139d
@devvratbhardwaj
model name modified
47cb601
@devvratbhardwaj
removed _client as POST call is directly being used
8111d09
@devvratbhardwaj
added print statements for debugging
a1ce2d7
@devvratbhardwaj
checking document lengths
42ffc29
@devvratbhardwaj
added \n for easier reading
5e73783
@devvratbhardwaj
text split len
618c17c
@devvratbhardwaj
Client call to RR instead of POST
aeb0dea
@devvratbhardwaj
removed redundant imports
6a757e5
@devvratbhardwaj
normalized the absolute relevance scores
3c7c997
@devvratbhardwaj
added support for batch processing
ea8cd1f
@devvratbhardwaj
updated print statements for better readability during debugging
b699c27
@devvratbhardwaj
Merge branch 'run-llama:main' into aimon-reranker
2254528
@devvratbhardwaj
Merge branch 'run-llama:main' into aimon-reranker
44b245c
@devvratbhardwaj
passing the pre-commit checks
be1d0e3
@devvratbhardwaj
removed print statements
44b5a55
@devvratbhardwaj
added unit tests and tests for batch processing
36acff9
@devvratbhardwaj
Added the AIMon reranker notebook to documentation.
bbc604a
@devvratbhardwaj
importing the svg image rather than rendering it.
793fe69
@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Mar 11, 2025
@devvratbhardwaj devvratbhardwaj marked this pull request as draft March 11, 2025 00:49
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
size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@devvratbhardwaj