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

fix(HelpdeskSearch): Add report to evaluate search and improvements with the same #1979

Merged
merged 10 commits into from
Sep 12, 2024
Merged

fix(HelpdeskSearch): Add report to evaluate search and improvements with the same #1979

merged 10 commits into from
Sep 12, 2024

Conversation

balamurali27
Copy link
Contributor

@balamurali27 balamurali27 commented Sep 12, 2024 •
edited
Loading

  • Add report to iteratively evaluate search

image

  • Add doctype for additional stopwords without adding it in code
    image

  • Search fixes

    • Use OR search for when AND search fails
    • Try noun phrase search before trying noun search
    • Search substrings instead of fuzzy (both together conflict). It's more intuitive
    • Clean input query better

@balamurali27
fix(HelpdeskSearch): Search for substring instead of fuzzy
57892ed 
Also return scores
@balamurali27
fix(HelpdeskSearch): Ignore ! and - in query
3cea9f4 
Split query string correctly. Ignore long whitespace
@balamurali27
fix(HelpdeskSearch): Add more fallbacks
a566ada 
- noun phrases
- or queries (worst case)
@balamurali27
fix(HelpdeskSearch): convert to lowercase before removing stopwords
69ec4d4 
capitalized stopwords weren't being removed

- Add more stopwords
- Add brown corpora to get noun phrases
@balamurali27
fix(HelpdeskSearch): Handle other exception
8187f8a
@balamurali27
feat(HelpdeskSearch): Add report to analyze tickets-search
6073377
@balamurali27
feat(HDSettings): Add property to get article search in server script
4ba371d
@balamurali27
feat(HelpdeskSearch): Use additional stopwords from doctype
88ffc70
@balamurali27
style: black + prettier
304476d
@balamurali27
fix(HelpdeskSearch): Only index published articles
3efd29f
@RitvikSardana RitvikSardana merged commit 4509a77 into develop Sep 12, 2024
2 checks passed
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@balamurali27 @RitvikSardana