e1da984e08
문제 SR과 카드 SR이 같은 간격 상수·산술을 참조하도록 순수함수 추출. 운영 동작 무변경.
- app/services/study/sr_schedule.py: REVIEW_INTERVAL_DAYS{1:3,2:7,3:14}/MASTERED=4/FIRST_DUE=1
+ advance(stage,outcome,now)→(new_stage,new_due) | None(skipped) + first_due(now).
진입 게이트(due_at IS NOT NULL/최초 due/skipped 불변)는 호출부 잔류(finalize vs review-complete 정책 차이).
- session_finalize.py: 상수·advance 분기 → sr_schedule import + sr_advance() (re-export 유지).
- study_question_progress.py: DEFAULT_FIRST_DUE_DAYS → sr_schedule import.
- 회귀 테스트 7/7: 전진 1·3·7·14·졸업·리셋·skipped불변·상수 + 전 stage×outcome 구 로직 바이트 동등.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
307 lines
12 KiB
Python
307 lines
12 KiB
Python
"""세션 종료 시 progress upsert + pattern 갱신 + 복습 stage 갱신 (Phase 1).
|
|
|
|
호출 시점: 마지막 attempt 가 박혀 status='done' 으로 전이된 직후.
|
|
한 트랜잭션 안에서 처리 — caller 가 commit 책임.
|
|
|
|
책임:
|
|
1. 이 세션의 attempts qid 별 그룹
|
|
2. 각 qid 별 progress upsert:
|
|
- last_outcome / last_attempted_at / last_attempt_id 갱신
|
|
- 최근 3회 attempts (이번 세션 + 과거) 로 pattern_state 계산
|
|
- pattern_window_attempts 갱신
|
|
3. 만약 progress.due_at IS NOT NULL (이미 복습 큐에 있던 문제):
|
|
- correct → review_stage += 1, due_at 다음 단계 (1→3→7→14일)
|
|
- stage ≥ 4 → due_at = NULL (학습완료)
|
|
- wrong/unsure → review_stage = 0, due_at = now()+1일
|
|
4. progress.due_at IS NULL 인 일반 풀이 결과는 due_at/stage 건드리지 않음
|
|
5. 세션 결과 요약 dict 반환
|
|
|
|
attempts 원본 수정 0건. progress upsert 만.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from collections import defaultdict
|
|
from dataclasses import dataclass
|
|
from datetime import datetime, timedelta, timezone
|
|
|
|
from sqlalchemy import select
|
|
from sqlalchemy.ext.asyncio import AsyncSession
|
|
|
|
from models.study_question import StudyQuestion, StudyQuestionAttempt
|
|
from models.study_question_progress import StudyQuestionProgress
|
|
from models.study_quiz_session import StudyQuizSession
|
|
from services.study.learning_pattern import (
|
|
PATTERN_CHRONIC_WRONG,
|
|
PATTERN_RECOVERED,
|
|
PATTERN_REGRESSED,
|
|
PATTERN_STABLE,
|
|
attempts_from_rows,
|
|
compute_pattern_state,
|
|
)
|
|
|
|
# SR 산술은 sr_schedule.py 단일 source (문제 SR + 카드 SR 공용). 상수는 재-export 유지.
|
|
from services.study.sr_schedule import ( # noqa: E402
|
|
DEFAULT_FIRST_DUE_DAYS, # noqa: F401
|
|
REVIEW_INTERVAL_DAYS, # noqa: F401
|
|
REVIEW_STAGE_MASTERED, # noqa: F401
|
|
advance as sr_advance,
|
|
)
|
|
|
|
|
|
@dataclass
|
|
class SessionSummary:
|
|
"""세션 결과 요약 — 응답 payload 직전 단계."""
|
|
correct: int
|
|
wrong: int
|
|
unsure: int
|
|
skipped: int
|
|
# 상태 변화 카운트
|
|
newly_correct: int # 처음 푸는데 맞힘 (이전 attempts 0건)
|
|
relapsed: int # 이전 정답이었으나 이번 wrong
|
|
recovered: int # pattern_state = recovered 새로 박힘
|
|
chronic_remaining: int # 이번 세션 후 chronic_wrong 으로 박힌 행 수
|
|
# 바로 할 일 후보
|
|
pending_review_count: int # wrong/unsure + 미확인 (last_reviewed_at < last_attempted_at)
|
|
chronic_count: int # pattern_state = chronic_wrong
|
|
regressed_count: int # pattern_state = regressed
|
|
|
|
|
|
async def finalize_session(
|
|
session: AsyncSession,
|
|
*,
|
|
user_id: int,
|
|
study_topic_id: int,
|
|
quiz_session_id: int,
|
|
) -> SessionSummary:
|
|
"""세션 종료 후처리. caller 가 commit. 멱등성 — 두 번 호출되어도 결과 같음."""
|
|
# 1. 이 세션 attempts 시간순
|
|
session_attempts = (
|
|
await session.execute(
|
|
select(StudyQuestionAttempt)
|
|
.where(
|
|
StudyQuestionAttempt.quiz_session_id == quiz_session_id,
|
|
StudyQuestionAttempt.user_id == user_id,
|
|
)
|
|
.order_by(StudyQuestionAttempt.answered_at.asc(), StudyQuestionAttempt.id.asc())
|
|
)
|
|
).scalars().all()
|
|
if not session_attempts:
|
|
return SessionSummary(0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0)
|
|
|
|
# qid 그룹: 한 세션 안에 같은 qid 가 두 번 박힐 가능성 거의 없지만 안전하게.
|
|
last_per_qid: dict[int, StudyQuestionAttempt] = {}
|
|
for a in session_attempts:
|
|
last_per_qid[a.study_question_id] = a # 시간 순 정렬이라 마지막이 최종
|
|
|
|
qids = list(last_per_qid.keys())
|
|
|
|
# 2. 기존 progress 행 일괄 fetch
|
|
existing = (
|
|
await session.execute(
|
|
select(StudyQuestionProgress)
|
|
.where(
|
|
StudyQuestionProgress.user_id == user_id,
|
|
StudyQuestionProgress.study_topic_id == study_topic_id,
|
|
StudyQuestionProgress.study_question_id.in_(qids),
|
|
)
|
|
)
|
|
).scalars().all()
|
|
progress_by_qid: dict[int, StudyQuestionProgress] = {p.study_question_id: p for p in existing}
|
|
|
|
# 3. qid 별 과거 attempts 일괄 fetch (이 세션 attempts 까지 포함, 시간순)
|
|
history_rows = (
|
|
await session.execute(
|
|
select(StudyQuestionAttempt)
|
|
.where(
|
|
StudyQuestionAttempt.user_id == user_id,
|
|
StudyQuestionAttempt.study_topic_id == study_topic_id,
|
|
StudyQuestionAttempt.study_question_id.in_(qids),
|
|
)
|
|
.order_by(
|
|
StudyQuestionAttempt.study_question_id.asc(),
|
|
StudyQuestionAttempt.answered_at.asc(),
|
|
StudyQuestionAttempt.id.asc(),
|
|
)
|
|
)
|
|
).scalars().all()
|
|
history_by_qid: dict[int, list[StudyQuestionAttempt]] = defaultdict(list)
|
|
for r in history_rows:
|
|
history_by_qid[r.study_question_id].append(r)
|
|
|
|
now = datetime.now(timezone.utc)
|
|
|
|
# 카운터 초기화
|
|
correct = wrong = unsure = skipped = 0
|
|
newly_correct = relapsed = recovered_count = 0
|
|
chronic_remaining = 0
|
|
|
|
for qid, last_attempt in last_per_qid.items():
|
|
outcome = last_attempt.outcome
|
|
if outcome == "correct":
|
|
correct += 1
|
|
elif outcome == "wrong":
|
|
wrong += 1
|
|
elif outcome == "unsure":
|
|
unsure += 1
|
|
elif outcome == "skipped":
|
|
skipped += 1
|
|
|
|
history = history_by_qid.get(qid, [])
|
|
# 이번 세션 이전 attempts 만 (status 변화 판단용)
|
|
prior = [a for a in history if a.id != last_attempt.id]
|
|
|
|
# 상태 변화 카운트
|
|
if not prior:
|
|
if outcome == "correct":
|
|
newly_correct += 1
|
|
else:
|
|
prior_last_outcome = prior[-1].outcome
|
|
if prior_last_outcome == "correct" and outcome == "wrong":
|
|
relapsed += 1
|
|
|
|
# pattern 계산
|
|
pattern_state, window_size = compute_pattern_state(attempts_from_rows(history))
|
|
if pattern_state == PATTERN_RECOVERED and (
|
|
qid not in progress_by_qid
|
|
or progress_by_qid[qid].pattern_state != PATTERN_RECOVERED
|
|
):
|
|
recovered_count += 1
|
|
if pattern_state == PATTERN_CHRONIC_WRONG:
|
|
chronic_remaining += 1
|
|
|
|
# progress upsert
|
|
progress = progress_by_qid.get(qid)
|
|
if progress is None:
|
|
progress = StudyQuestionProgress(
|
|
user_id=user_id,
|
|
study_topic_id=study_topic_id,
|
|
study_question_id=qid,
|
|
)
|
|
session.add(progress)
|
|
progress_by_qid[qid] = progress
|
|
|
|
progress.last_outcome = outcome
|
|
progress.last_attempted_at = last_attempt.answered_at
|
|
progress.last_attempt_id = last_attempt.id
|
|
progress.pattern_state = pattern_state
|
|
progress.pattern_updated_at = now
|
|
progress.pattern_window_attempts = window_size
|
|
|
|
# 복습 stage 갱신 — 이미 due_at 박힌 문제만 (산술은 sr_schedule 공용)
|
|
if progress.due_at is not None:
|
|
result = sr_advance(progress.review_stage, outcome, now)
|
|
if result is not None: # skipped 는 None → due_at/stage 불변
|
|
progress.review_stage, progress.due_at = result
|
|
# progress.due_at IS NULL 일반 풀이 → stage 건드리지 않음
|
|
|
|
# 4. 바로 할 일 카운트 (요약 응답용) — finalize 직후 progress 상태 기준 SQL 한 번
|
|
pending_review_count = await _count_pending_review(session, user_id, study_topic_id)
|
|
chronic_count = await _count_pattern(session, user_id, study_topic_id, PATTERN_CHRONIC_WRONG)
|
|
regressed_count = await _count_pattern(session, user_id, study_topic_id, PATTERN_REGRESSED)
|
|
|
|
# 5. quiz_session row 에 스냅샷 영속화 (Phase 2-B). 결과 화면 헤더 노출 + 멱등.
|
|
qs = await session.get(StudyQuizSession, quiz_session_id)
|
|
if qs is not None:
|
|
qs.newly_correct_count = newly_correct
|
|
qs.relapsed_count = relapsed
|
|
qs.recovered_count = recovered_count
|
|
qs.chronic_remaining_count = chronic_remaining
|
|
|
|
# 6. Phase 4-A: 이 세션의 wrong/unsure qid AI 풀이 prefetch enqueue (best-effort).
|
|
# 실패가 finalize 자체를 깨뜨리지 않도록 try/except 로 격리. 응답 카운트와 무관.
|
|
try:
|
|
from services.study.explanation_enqueue import enqueue_explanation_for_qids
|
|
wrong_unsure_qids = [
|
|
qid for qid, a in last_per_qid.items()
|
|
if a.outcome in ("wrong", "unsure")
|
|
]
|
|
if wrong_unsure_qids:
|
|
res = await enqueue_explanation_for_qids(
|
|
session, user_id=user_id, qids=wrong_unsure_qids,
|
|
)
|
|
import logging
|
|
logging.getLogger(__name__).info(
|
|
"phase4a_finalize_enqueue session=%s candidates=%s enqueued=%s skipped=%s",
|
|
quiz_session_id, res["candidate_count"], res["enqueue_count"], res["skipped_count"],
|
|
)
|
|
except Exception as e:
|
|
import logging
|
|
logging.getLogger(__name__).warning(
|
|
"phase4a_finalize_enqueue_failed session=%s: %s: %s",
|
|
quiz_session_id, type(e).__name__, e,
|
|
)
|
|
|
|
# 7. Phase 4-B v1: 세션 단위 분석 enqueue (best-effort).
|
|
# wrong/unsure < 5 면 worker 가 insufficient_attempts 로 skipped — finalize 는 무관.
|
|
try:
|
|
from services.study.session_analysis_enqueue import enqueue_session_analysis_auto
|
|
res = await enqueue_session_analysis_auto(
|
|
session, user_id=user_id, study_quiz_session_id=quiz_session_id,
|
|
)
|
|
import logging
|
|
logging.getLogger(__name__).info(
|
|
"phase4b_finalize_enqueue session=%s enqueued=%s",
|
|
quiz_session_id, res["enqueued"],
|
|
)
|
|
except Exception as e:
|
|
import logging
|
|
logging.getLogger(__name__).warning(
|
|
"phase4b_finalize_enqueue_failed session=%s: %s: %s",
|
|
quiz_session_id, type(e).__name__, e,
|
|
)
|
|
|
|
return SessionSummary(
|
|
correct=correct,
|
|
wrong=wrong,
|
|
unsure=unsure,
|
|
skipped=skipped,
|
|
newly_correct=newly_correct,
|
|
relapsed=relapsed,
|
|
recovered=recovered_count,
|
|
chronic_remaining=chronic_remaining,
|
|
pending_review_count=pending_review_count,
|
|
chronic_count=chronic_count,
|
|
regressed_count=regressed_count,
|
|
)
|
|
|
|
|
|
async def _count_pending_review(session: AsyncSession, user_id: int, topic_id: int) -> int:
|
|
"""pending_review 탭 조건 기반 카운트."""
|
|
from sqlalchemy import and_, func, or_
|
|
row = await session.execute(
|
|
select(func.count())
|
|
.select_from(StudyQuestionProgress)
|
|
.where(
|
|
StudyQuestionProgress.user_id == user_id,
|
|
StudyQuestionProgress.study_topic_id == topic_id,
|
|
StudyQuestionProgress.last_outcome.in_(("wrong", "unsure")),
|
|
or_(
|
|
StudyQuestionProgress.last_reviewed_at.is_(None),
|
|
and_(
|
|
StudyQuestionProgress.last_reviewed_at.is_not(None),
|
|
StudyQuestionProgress.last_attempted_at.is_not(None),
|
|
StudyQuestionProgress.last_reviewed_at
|
|
< StudyQuestionProgress.last_attempted_at,
|
|
),
|
|
),
|
|
)
|
|
)
|
|
return int(row.scalar() or 0)
|
|
|
|
|
|
async def _count_pattern(
|
|
session: AsyncSession, user_id: int, topic_id: int, pattern: str
|
|
) -> int:
|
|
from sqlalchemy import func
|
|
row = await session.execute(
|
|
select(func.count())
|
|
.select_from(StudyQuestionProgress)
|
|
.where(
|
|
StudyQuestionProgress.user_id == user_id,
|
|
StudyQuestionProgress.study_topic_id == topic_id,
|
|
StudyQuestionProgress.pattern_state == pattern,
|
|
)
|
|
)
|
|
return int(row.scalar() or 0)
|