Créneaux de nuit rattachés à leur journée de festival
Un créneau saisi « 00:00–01:00 » sur la carte du samedi était enregistré le samedi à 00:00 (nuit de vendredi à samedi), donc affiché la veille dans la grille des besoins. Une heure de début avant le début de journée (premier créneau type) désigne désormais la nuit qui suit : dimanche 00:00, à la suite de la soirée du samedi. Même « journée de festival » partout (validation/engine.py:day_boundary, festival_day ; miroir front lib/planning.ts:festivalDay) : écran Postes & créneaux, feuilles de poste (une seule page pour une soirée qui passe minuit) et chaînage du solveur (00h–01h enchaîne 23h–00h). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
AdamThomas
co-authored by
Claude Opus 5.5
Claude Opus 5.5
parent
c60bc11795
commit
a6e7b6e7fb
@@ -17,7 +17,7 @@ from app.api.deps import get_edition
|
||||
from app.api.schemas import PositionSheetsRequest
|
||||
from app.auth.deps import require_coordinator
|
||||
from app.db import get_session
|
||||
from app.models import Account, Artist, Assignment, Edition, Position, Shift, Volunteer
|
||||
from app.models import Account, Artist, Assignment, DayPart, Edition, Position, Shift, Volunteer
|
||||
from app.sheets.pdf import (
|
||||
PositionDaySheet,
|
||||
SheetArtist,
|
||||
@@ -25,6 +25,7 @@ from app.sheets.pdf import (
|
||||
SheetVolunteer,
|
||||
generate_position_sheets_pdf,
|
||||
)
|
||||
from app.validation.engine import day_boundary, festival_day
|
||||
|
||||
router = APIRouter(
|
||||
prefix="/editions/{edition_id}/position-sheets", tags=["position-sheets"]
|
||||
@@ -69,6 +70,13 @@ def generate_position_sheets(
|
||||
for vs in volunteers_by_shift.values():
|
||||
vs.sort(key=lambda v: (v.last_name, v.first_name))
|
||||
|
||||
# « Jour » = journée de festival (CLAUDE.md § 3), pas date du calendrier :
|
||||
# une soirée qui passe minuit (23h–00h, 00h–01h…) reste sur une seule
|
||||
# feuille, celle du jour où elle a commencé.
|
||||
boundary = day_boundary(
|
||||
session.exec(select(DayPart).where(DayPart.edition_id == edition.id))
|
||||
)
|
||||
|
||||
# Un jour sans aucun besoin réel (créneau 0/0 généré automatiquement,
|
||||
# jamais rempli) n'apparaît pas — évite des pages « personne affecté »
|
||||
# pour des créneaux qui ne représentent aucun besoin.
|
||||
@@ -77,7 +85,8 @@ def generate_position_sheets(
|
||||
undefined_need = shift.min_staff == 0 and shift.ideal_staff == 0
|
||||
if undefined_need and not volunteers_by_shift.get(shift.id):
|
||||
continue
|
||||
shifts_by_position_day[(shift.position_id, shift.starts_at.date())].append(shift)
|
||||
day = festival_day(shift.starts_at, boundary)
|
||||
shifts_by_position_day[(shift.position_id, day)].append(shift)
|
||||
|
||||
Referent = aliased(Volunteer)
|
||||
referent_rows = session.exec(
|
||||
|
||||
@@ -10,6 +10,7 @@ from app.models import (
|
||||
Artist,
|
||||
Assignment,
|
||||
Availability,
|
||||
DayPart,
|
||||
Edition,
|
||||
Shift,
|
||||
Volunteer,
|
||||
@@ -18,6 +19,7 @@ from app.models import (
|
||||
VolunteerPositionWish,
|
||||
)
|
||||
from app.solver.models import SolverInput, SolverShift, SolverVolunteer
|
||||
from app.validation.engine import day_boundary, festival_day
|
||||
from app.validation.models import AvailabilityWindow, ShiftInfo
|
||||
|
||||
|
||||
@@ -41,6 +43,11 @@ def build_solver_input(*, session: Session, edition: Edition) -> SolverInput:
|
||||
)
|
||||
shifts_by_id = {s.id: s for s in shifts}
|
||||
|
||||
# « Même jour » = même journée de festival (CLAUDE.md § 3) : 00h–01h
|
||||
# enchaîne bien 23h–00h de la veille au calendrier.
|
||||
boundary = day_boundary(
|
||||
session.exec(select(DayPart).where(DayPart.edition_id == edition.id))
|
||||
)
|
||||
# Prédécesseur(s) : créneaux (n'importe quel poste) du même jour dont la
|
||||
# fin coïncide avec le début de celui-ci — sert au malus de chaînage
|
||||
# (voir `engine._score`), pour limiter les enchaînements de shifts plutôt
|
||||
@@ -49,7 +56,7 @@ def build_solver_input(*, session: Session, edition: Edition) -> SolverInput:
|
||||
predecessor_ids: dict = {}
|
||||
by_day: dict = defaultdict(list)
|
||||
for s in shifts:
|
||||
by_day[s.starts_at.date()].append(s)
|
||||
by_day[festival_day(s.starts_at, boundary)].append(s)
|
||||
for day_shifts in by_day.values():
|
||||
ids_by_end: dict = defaultdict(list)
|
||||
for s in day_shifts:
|
||||
|
||||
@@ -107,26 +107,34 @@ def _check_outside_availability(data: ValidationInput) -> list[Violation]:
|
||||
# --------------------------------------------------------------------------- #
|
||||
# DAILY_HOURS_EXCEEDED (avertissement) #
|
||||
# --------------------------------------------------------------------------- #
|
||||
def _day_boundary(day_parts: tuple) -> dt.time:
|
||||
def day_boundary(day_parts) -> dt.time:
|
||||
"""Début d'une « journée de festival » : début du premier créneau type
|
||||
(`DayPart`, par `order`), minuit à défaut — voir CLAUDE.md § 3. Accepte
|
||||
aussi bien les `DayPartInfo` de ce module que les lignes `DayPart`."""
|
||||
day_parts = list(day_parts)
|
||||
if not day_parts:
|
||||
return dt.time(0, 0)
|
||||
return min(day_parts, key=lambda part: part.order).start_time
|
||||
|
||||
|
||||
def _day_key(moment: dt.datetime, boundary: dt.time) -> dt.date:
|
||||
def festival_day(moment: dt.datetime, boundary: dt.time) -> dt.date:
|
||||
"""Journée de festival d'un instant : avant `boundary`, c'est encore la
|
||||
nuit de la veille (ex. 01:00 le dimanche ⇒ samedi). Partagé avec le
|
||||
solveur et les feuilles de poste, pour qu'une soirée qui passe minuit
|
||||
reste d'un seul tenant partout."""
|
||||
if moment.time() >= boundary:
|
||||
return moment.date()
|
||||
return moment.date() - dt.timedelta(days=1)
|
||||
|
||||
|
||||
def _check_daily_hours(data: ValidationInput) -> list[Violation]:
|
||||
boundary = _day_boundary(data.day_parts)
|
||||
boundary = day_boundary(data.day_parts)
|
||||
candidate = data.candidate_shift
|
||||
key = _day_key(candidate.starts_at, boundary)
|
||||
key = festival_day(candidate.starts_at, boundary)
|
||||
|
||||
total = candidate.ends_at - candidate.starts_at
|
||||
for other in data.existing_shifts:
|
||||
if _day_key(other.starts_at, boundary) == key:
|
||||
if festival_day(other.starts_at, boundary) == key:
|
||||
total += other.ends_at - other.starts_at
|
||||
|
||||
if total > _EIGHT_HOURS:
|
||||
|
||||
@@ -149,3 +149,28 @@ def test_position_ids_filter_scopes_the_sheets(client, edition, session):
|
||||
|
||||
resp = client.post(_url(edition), json={"position_ids": [str(welcome.id)]})
|
||||
assert resp.status_code == 200
|
||||
|
||||
|
||||
def test_a_night_that_passes_midnight_stays_on_its_festival_day(client, edition, session):
|
||||
# Début de journée = 06:00 (premier créneau type de la fixture) : 00h–02h
|
||||
# le 10 appartient encore à la soirée du 9 — une seule page, pas deux.
|
||||
from io import BytesIO
|
||||
|
||||
from pypdf import PdfReader
|
||||
|
||||
position = make_position(session, edition, name="Bar de nuit")
|
||||
for start, end in [
|
||||
(dt_at(9, 22), dt_at(9, 23)),
|
||||
(dt_at(9, 23), dt_at(10, 0)),
|
||||
(dt_at(10, 0), dt_at(10, 1)),
|
||||
(dt_at(10, 1), dt_at(10, 2)),
|
||||
]:
|
||||
make_shift(session, edition, position, start=start, end=end)
|
||||
|
||||
resp = client.post(_url(edition), json={})
|
||||
assert resp.status_code == 200
|
||||
reader = PdfReader(BytesIO(resp.content))
|
||||
assert len(reader.pages) == 1
|
||||
text = reader.pages[0].extract_text()
|
||||
assert "jeudi 9 juillet" in text.lower()
|
||||
assert "01:00" in text
|
||||
|
||||
@@ -0,0 +1,27 @@
|
||||
"""Construction du `SolverInput` depuis la base (`app/solver/loader.py`)."""
|
||||
|
||||
from app.solver.loader import build_solver_input
|
||||
from tests.factories import dt_at, make_position, make_shift
|
||||
|
||||
|
||||
def test_predecessor_is_found_across_midnight_within_the_festival_day(edition, session):
|
||||
# Début de journée = 06:00 (fixture) : 23h–00h le 9 puis 00h–01h le 10
|
||||
# appartiennent à la même soirée — le second enchaîne le premier.
|
||||
bar = make_position(session, edition, name="Bar")
|
||||
before = make_shift(session, edition, bar, start=dt_at(9, 23), end=dt_at(10, 0))
|
||||
after = make_shift(session, edition, bar, start=dt_at(10, 0), end=dt_at(10, 1))
|
||||
|
||||
data = build_solver_input(session=session, edition=edition)
|
||||
by_id = {s.id: s for s in data.shifts}
|
||||
assert by_id[str(after.id)].predecessor_shift_ids == frozenset({str(before.id)})
|
||||
|
||||
|
||||
def test_no_predecessor_across_the_festival_day_boundary(edition, session):
|
||||
# 05h–06h le 10 clôt la journée du 9 ; 06h–07h le 10 ouvre la suivante.
|
||||
bar = make_position(session, edition, name="Bar")
|
||||
make_shift(session, edition, bar, start=dt_at(10, 5), end=dt_at(10, 6))
|
||||
morning = make_shift(session, edition, bar, start=dt_at(10, 6), end=dt_at(10, 7))
|
||||
|
||||
data = build_solver_input(session=session, edition=edition)
|
||||
by_id = {s.id: s for s in data.shifts}
|
||||
assert not by_id[str(morning.id)].predecessor_shift_ids
|
||||
Reference in New Issue
Block a user