Skip to content

Conversation

@abeglova
Copy link
Contributor

@abeglova abeglova commented Jul 8, 2024

What are the relevant tickets?

closes https://github.com/mitodl/hq/issues/4799

Description (What does it do?)

This PR updates the weight of instructor names in the search query so that searching by instructor works better

How can this be tested?

If you do not already have Eric Grimson's ocw courses in your search index run

docker-compose run web ./manage.py backpopulate_ocw_data --course-name 6-001-structure-and-interpretation-of-computer-programs-spring-2005 --skip-content-files
docker-compose run web ./manage.py backpopulate_ocw_data --course-name 6-00-introduction-to-computer-science-and-programming-fall-2008 --skip-content-files
docker-compose run web ./manage.py backpopulate_ocw_data --course-name 6-0002-introduction-to-computational-thinking-and-data-science-fall-2016 --skip-content-files
docker-compose run web ./manage.py backpopulate_ocw_data --course-name 6-0001-introduction-to-computer-science-and-programming-in-python-fall-2016 --skip-content-files

Go to http://localhost:8062/search/?q=Eric+Grimson

Verify that the four ocw courses above are the first four results

@abeglova abeglova marked this pull request as ready for review July 8, 2024 19:51
@mbertrand mbertrand self-assigned this Jul 9, 2024
Copy link
Member

@mbertrand mbertrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great 👍

@abeglova abeglova merged commit 0d15f55 into main Jul 9, 2024
@odlbot odlbot mentioned this pull request Jul 10, 2024
27 tasks
mbertrand pushed a commit that referenced this pull request Jul 15, 2024
@rhysyngsun rhysyngsun deleted the ab/add-extra-weight-to-instructor-last-name branch February 7, 2025 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants