-
Notifications
You must be signed in to change notification settings - Fork 11
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.
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
Использование API для получения макс. балла для закрытых контестов #109
Conversation
@@ -715,6 +720,17 @@ def update_cache(full_score, run_penalty, current_penalty, deadlines): | |||
elif key.text == 'Deadline:': | |||
deadlines.hard = datetime.strptime(value.text, '%Y/%m/%d %H:%M:%S') | |||
|
|||
# TODO check API status for running / testing kr contests | |||
if not full_score_found: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Используется такое условие вместо past_deadline
, потому что:
- После дедлайна у задачи могут отображаться баллы (и вся остальная информация). Например, sm01, задачи из закрытых контестов, у которых есть реджекты
- Судя по старому комментарию, даже у задачи с ещё не прошедшим дедлайном (sm01-3) может отсутствовать информация о макс. балле. В то же время через API его (должно быть) возможно получить
try: | ||
problem_status = session.api().problem_status(problem.extract_id()).get('problem', {}) | ||
except APIError as e: | ||
click.secho(f'Cannot get problem info ({problem.short_name}): {e}', err=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Такого происходить, по идее, не должно.
Во время проведения КР1 и КР2, когда задачи даже не отображались в списке, через API нельзя было получить условие. Возможно, в это время могла возвращаться какая-то ошибка при вызове метода problem_status
Максимальный балл для КР сейчас все равно не используется, поэтому, если через API можно получить корректное значение, его обработку можно будет добавить позже |
См #106