Skip to content

Add query preprocessing before hybrid retrieval - #175

Merged
SharanyaAchanta merged 5 commits into
SharanyaAchanta:mainfrom
Shalini828:feature/query-preprocessing
Mar 3, 2026
Merged

Add query preprocessing before hybrid retrieval#175
SharanyaAchanta merged 5 commits into
SharanyaAchanta:mainfrom
Shalini828:feature/query-preprocessing

Conversation

@Shalini828

Copy link
Copy Markdown
Contributor

Summary

This PR introduces a query preprocessing step before hybrid retrieval.

Changes

  • Added engine/preprocessing.py
  • Implemented preprocess_query() to normalize user input
  • Integrated preprocessing into search_pdfs() before retrieval logic

Why

This ensures consistent query formatting and improves retrieval quality by:

  • Lowercasing queries
  • Removing extra whitespace
  • Standardizing input before embedding and keyword search

Impact

Improves robustness of hybrid search pipeline without affecting existing logic.

@github-actions

Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@centiceron centiceron added the On hold Temporary "on hold" status for maintainence label Feb 27, 2026
@SharanyaAchanta

Copy link
Copy Markdown
Owner

@Shalini828
Thanks for the PRs. I was away for a few days, so just catching up now.
There are some merge conflicts because of the recent updates. If you have some time, could you please resolve them according to the current architecture? Since we are approaching the project wrap-up,so we’ll need to wrap this up soon. If you're up for the challenge, go for it! All the best. If not, no worries at all thanks again for your contribution!

@Shalini828

Copy link
Copy Markdown
Contributor Author

I've updated the PR according to the latest architecture and resolved the CI issues. All checks are now passing. Please let me know if anything else needs improvement.

@SharanyaAchanta

Copy link
Copy Markdown
Owner

Hi @Shalini828 Shalini, thanks for the update and for fixing the CI issues!

Just a heads-up: I merged another PR recently which caused some new conflicts in this branch. Since today is the final day of the program, I'm trying to wrap everything up quickly.

If you're able to resolve these new conflicts right away, please go ahead and I’ll check it. But if it’s taking too much time, no worries at all—you don't have to stress about it since we're on a tight deadline. Let me know what you think!

@SharanyaAchanta

Copy link
Copy Markdown
Owner

@Shalini828 Now the conflicts are resolved great job. But some checks failed please check that if you are free.

@Shalini828

Copy link
Copy Markdown
Contributor Author

I've checked and the failing tests are resolved now. All checks are passing.

@SharanyaAchanta

Copy link
Copy Markdown
Owner

I've checked and the failing tests are resolved now. All checks are passing.

Great job Shalini

@SharanyaAchanta

Copy link
Copy Markdown
Owner

By the way @Shalini828 can you please link the issue number too.

@SharanyaAchanta
SharanyaAchanta merged commit 7bf74f1 into SharanyaAchanta:main Mar 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Hard On hold Temporary "on hold" status for maintainence OSCG26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants