Skip to content

Weighted mark#100

Merged
Temmmmmo merged 3 commits intomainfrom
weighted-mark
Feb 22, 2025
Merged

Weighted mark#100
Temmmmmo merged 3 commits intomainfrom
weighted-mark

Conversation

@DaymasS
Copy link
Copy Markdown
Contributor

@DaymasS DaymasS commented Feb 18, 2025

Изменения

Добавлена релевантность оценок

Детали реализации

Взвешенная оценка рассчитывается по формуле:
(оценка преподавателя * кол-во комментариев + средняя оценка * константа) / (кол-во комментариев + константа)
Константу взял за 0.5. При увелечении взвешенная оценка лекторов с меньшим кол-вом комментариев будет сильнее стремиться к средней оценке по всем лекторам

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@DaymasS DaymasS requested a review from Temmmmmo February 18, 2025 06:20
@DaymasS DaymasS self-assigned this Feb 18, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 18, 2025

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
rating_api
   __main__.py440%1–7
   exceptions.py34779%35–37, 48–50, 58
rating_api/models
   base.py55591%24–27, 76
   db.py85396%61, 75, 84
rating_api/routes
   base.py26485%48–50, 52
   comment.py1012278%34, 38, 56, 74, 109–111, 132–141, 184, 194–199, 206, 230, 246
   exc_handlers.py23387%33, 40, 47
   lecturer.py891682%136, 141–167, 183, 191, 209, 215
rating_api/schemas
   base.py12467%6–9
rating_api/utils
   logging_utils.py432542%21–42, 51, 58–64, 70–83
TOTAL5909384% 

Summary

Tests Skipped Failures Errors Time
40 0 💤 0 ❌ 0 🔥 7.886s ⏱️

@DaymasS DaymasS linked an issue Feb 18, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Member

@Temmmmmo Temmmmmo left a comment

Choose a reason for hiding this comment

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

в целом норм

Comment thread rating_api/models/db.py
@Temmmmmo Temmmmmo merged commit 76a3805 into main Feb 22, 2025
@DaymasS DaymasS deleted the weighted-mark branch March 14, 2025 00:59
@DaymasS DaymasS restored the weighted-mark branch March 14, 2025 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Релевантность отзывов

2 participants