Skip to content

Conversation

@mbertrand
Copy link
Member

@mbertrand mbertrand commented Aug 22, 2024

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/5258

Description (What does it do?)

  • Adds a completeness field to LearningResource with a default value of 1.0
  • Calculates the value for OCW courses during ETL contentfile ingestion

How can this be tested?

  • Copy OCW_ settings from RC to your backend env file and also set SKIP_TIKA=True
  • Run docker compose up, a migration should complete
  • Run the following mgmt commands (do NOT use the --skip-contentfiles option):
    ./manage.py backpopulate_ocw_data --course-name 6-622-power-electronics-spring-2023
    ./manage.py backpopulate_ocw_data --course-name 6-801-machine-vision-fall-2020
    ./manage.py backpopulate_ocw_data --course-name sp-248-neet-ways-of-thinking-fall-2023
    
  • When done, the courses should have a completeness score of 1.0, 0.8, and 0.033333333
  • Try the new management command with several different options, they should complete without errors.
    ./manage.py backpopulate_ocw_scores --course-name  6-801-machine-vision-fall-2020
    ./manage.py backpopulate_ocw_scores  --csv --skip_calc scores.csv
    

@mbertrand mbertrand changed the title Completeness score OCW course completeness score Aug 23, 2024
@mbertrand mbertrand added Needs Review An open Pull Request that is ready for review and removed Work in Progress labels Aug 27, 2024
@mbertrand mbertrand requested a review from gumaerc August 28, 2024 13:33
@abeglova abeglova self-assigned this Aug 28, 2024
Copy link
Contributor

@abeglova abeglova left a comment

Choose a reason for hiding this comment

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

LGTM - although it looks like a lot of ocw courses have very low completeness scores. We will need to figure out how much to penalize courses with scores close to zero in the results

@mbertrand mbertrand merged commit 811d7b8 into main Aug 28, 2024
@odlbot odlbot mentioned this pull request Aug 29, 2024
4 tasks
@rhysyngsun rhysyngsun deleted the mb/complete_score branch February 7, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review An open Pull Request that is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants