diff --git a/AGENTS.md b/AGENTS.md index f9adb9c..6f43538 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -24,6 +24,7 @@ way; this file is *how to work in them*. | login, who owns a school, watching others | [`docs/design/session.md`](docs/design/session.md) — slice 8, phases 38–39 | | clock speed buttons and high-speed stride | `src/HSchool.Simulation/ClockSpeed.cs` **and** [`docs/design/session.md`](docs/design/session.md) — phase 40 | | talks, opinions, fights, romance pack | [`docs/design/social.md`](docs/design/social.md) — slice 9, phases 41–47 | +| lesson consequences (teacher present, skill, warmth, textbook, weather commute) | [`docs/design/off-queue.md`](docs/design/off-queue.md) — phases 48, 50–53 | | a task that is not in any slice | `.claude/skills/side-work` — discuss, document in «Вне очереди» or a new named slice, then implement | ## Commands diff --git a/docs/design/off-queue.md b/docs/design/off-queue.md new file mode 100644 index 0000000..f0661fc --- /dev/null +++ b/docs/design/off-queue.md @@ -0,0 +1,90 @@ +# Вне очереди + +Задачи, которых нет в нумерованных срезах. Сюда не дописывают срез 9 и не заводят срез 10: +это склейка уже живых систем, после любой фазы школа целая. + +Люди: [`people.md`](people.md). Штат: [`staffing.md`](staffing.md). Расписание: +[`schedule.md`](schedule.md). Присутствие: [`ai.md`](ai.md). Одежда и климат: +[`inventory.md`](inventory.md). + +## Связность урока + +Урок уже растёт навык, если ученик стоит в кабинете и не голоден. Рядом лежат числа, которые +игрок видит и которые на этот рост не влияют: учитель в туалете, навык учителя, тепло, +учебник в сумке, снег на дороге. + +Этот кусок отвечает на вопрос «почему нанимать сильного, чинить тепло и класть учебник в +сумку». Оценок, посещаемости как журнала и приказов нет. + +### Что меняется в уже принятом + +| Было | Стало | Почему | +| --- | --- | --- | +| Ученик в кабинете качает навык, учитель может быть где угодно | Рост только если назначенный учитель стоит в том же узле и не идёт | Зазор «обязанность / место» уже есть; урок его не смотрел | +| Зарплата зависит от навыков, урок — нет | Навык учителя по предмету масштабирует `Gain` | Иначе «математика 78» в пуле — только цена | +| Голод режет урок, тепло — нет | Та же `NeedFactor` на тепло | Полоска уже живая; [`inventory.md`](inventory.md) откладывал штраф | +| Учебники сегодняшних предметов едут в сумке | Нет в сумке на этом уроке — половинный рост | Переноска и шкафчик иначе остаются двумя числами | +| Погода одевает, на приход не влияет | Дождь и снег добавляют минуты к запасу на дорогу | Часы уже показывают осадки; ленивый уже опаздывает от черты | + +### Формула + +`HSchool.Ai.LessonLearning.Gain` по-прежнему чистое число. Множители: + +- **нужды** — `NeedFactor(голод) × NeedFactor(тепло)`; нет нужды в каталоге — множитель 1; +- **черты ученика** — как сейчас; +- **учитель** — `NeedFactor(навык / 100)`, навык — среднее по `SubjectDef.skills` у учителя, + нет ключа — `Range.Min` этого навыка; учителя нет в комнате — множитель 0, рост не пишется; +- **учебник** — 1, если в сумке есть экземпляр с `subject` этого урока; иначе + `BehaviorDef.lessonNoTextbookFactor` (ваниль 0.5). + +Учитель «стоит в комнате»: `NodeId` = кабинет урока, путь пуст, остаток минут 0. Действие +учителя не отменяет урок (дежурство — не `ActionDef`). Нет человека с этим id — учителя нет. + +Дорога: к нынешнему `SlackMinutes` (черты + бросок) прибавляются минуты из +`BehaviorDef.commuteRainMinutes` / `commuteSnowMinutes`, если в момент сборки плана дня на +улице дождь или снег. `HSchool.Ai` осадки не знает — симуляция передаёт уже посчитанные минуты. + +### Что видит игрок + +Числа на карточке. Плюс строка в сегодняшней истории, **один раз на ключ в сутки**, не каждый +тик: + +| Тип | Когда | `thingDef` | +| --- | --- | --- | +| `lesson-no-teacher` | Урок шёл, учителя не было в кабинете | предмет | +| `lesson-cold` | На уроке, который иначе учил, тепло ниже `needThreshold` | предмет | +| `lesson-no-textbook` | На уроке нет учебника этого предмета в сумке | предмет | + +Опоздание из-за снега в лог не пишем: весь состав получил бы одну и ту же строку, а место +человека и так видно. + +Клиент по-прежнему рисует `label` с сервера. Сокет и сейв не меняются. Версию протокола не +бампить. + +### Где живёт код + +Нового проекта нет. Формула и план дня — `HSchool.Ai`. Присутствие учителя, тепло, сумка, +погода на входе в план — `HSchool.Simulation`. Подписи лога — каталог. Список типов — +`docs/protocol.md` (HTTP лога). + +### Заведомо не сейчас + +- Оценки, журнал посещаемости, выговор за прогул. +- Перегруз замедляет ходьбу. +- Насмешки над формой и мнения — срез 9. +- Вызов родителей, деньги за одежду, болезнь от промокания. +- Клиент сам считает «учится хуже». + +### Зафиксировано этим разговором + +| Тема | Решение | +| --- | --- | +| Нарезка | Пять фаз во «Вне очереди», не срез 10 | +| Стоп | После каждой фазы школа целая | +| Протокол | Не бампить; новые `type` только в HTTP лога | +| Сейв | Не трогать | +| Учитель на уроке | Тот же узел, не идёт; нет id — нет роста | +| Навык учителя | Среднее по скиллам предмета, та же кривая что у нужды | +| Тепло | Рядом с голодом, произведение двух `NeedFactor` | +| Учебник | Только сумка, не шкафчик и не дом; ваниль ×0.5 | +| Погода на дорогу | Минуты в `BehaviorDef`; лога нет | diff --git a/docs/phases/48-lesson-teacher-present.md b/docs/phases/48-lesson-teacher-present.md new file mode 100644 index 0000000..9c19b42 --- /dev/null +++ b/docs/phases/48-lesson-teacher-present.md @@ -0,0 +1,43 @@ +# Фаза 48. Учитель в кабинете + +## Зависимости + +- [Фаза 21](21-decisions.md) +- [Фаза 18](18-presence-walking.md) + +## Зачем + +Ученик в кабинете больше не качает навык в пустоту. Нет учителя на месте — нет роста. + +## Задачи + +- [x] `LessonLearningSystem` растёт навык, только если назначенный учитель стоит в + `lesson.RoomId`: на кампусе, путь пуст, остаток минут 0 +- [x] Нет человека с `TeacherId` в мире — как отсутствующий +- [x] Действие учителя урок не отменяет +- [x] Формула `LessonLearning.Gain` не меняется +- [x] Лог ученика `lesson-no-teacher` один раз на предмет в сутки, не каждый тик; `thingDef` — + предмет; подпись из каталога +- [x] `docs/protocol.md` — новый `type` у HTTP лога, без бампа сокета +- [x] Фикстуры, где учитель был строкой `t1` вне ростера, нанимают живого человека — иначе + существующий тест роста навыка зелёный впустую + +## Тесты, без которых фаза не закрыта + +- [x] Ученик и учитель стоят в кабинете → навык за урок вырос +- [x] Ученик в кабинете, учитель в другом узле → навык не вырос +- [x] Учитель идёт по коридору в этот кабинет → пока идёт, роста нет +- [x] Нет человека с id учителя → роста нет +- [x] За урок без учителя в логе ученика одна строка `lesson-no-teacher`, не по числу тиков +- [x] Голодный по-прежнему учится хуже сытого, когда учитель на месте + +## Критерий готовности + +- Учитель ушёл в туалет посреди математики — класс в кабинете есть, навык за эти минуты не + ползёт +- `dotnet test` проходит + +## Стоп + +Не масштабировать рост навыком учителя. Не трогать тепло, учебник и дорогу. Не бампить +протокол. Не писать лог учителю. diff --git a/docs/phases/50-teacher-skill-gain.md b/docs/phases/50-teacher-skill-gain.md new file mode 100644 index 0000000..01cd6c4 --- /dev/null +++ b/docs/phases/50-teacher-skill-gain.md @@ -0,0 +1,34 @@ +# Фаза 50. Навык учителя на уроке + +## Зависимости + +- [Фаза 48](48-lesson-teacher-present.md) +- [Фаза 12](12-hiring-budget.md) + +## Зачем + +Соискатель с математикой 78 перестаёт отличаться от 41 только зарплатой. + +## Задачи + +- [ ] `LessonLearning.TeacherFactor` — та же кривая, что `NeedFactor`: навык 0 → 0.25, 100 → 1 +- [ ] Навык учителя — среднее его значений по `SubjectDef.skills`; нет ключа — `Range.Min` +- [ ] `Gain` умножает на этот фактор; учитель в кабинете по-прежнему обязателен (фаза 48) +- [ ] Числа кривой в коде рядом с нуждой, не новый экран и не сейв +- [ ] Лог не пишем: рост тише — это и есть сигнал + +## Тесты, без которых фаза не закрыта + +- [ ] Учитель с навыком 20 даёт меньший прирост, чем с 80, при том же ученике и голоде +- [ ] Предмет с несколькими скиллами берёт среднее, а не первый попавшийся в словаре +- [ ] Нет ключа навыка у учителя — подставляется минимум шкалы, рост не падает в ноль +- [ ] Учителя нет в кабинете — по-прежнему нулевой рост, фактор не спасает + +## Критерий готовности + +- Два учителя математики в тестовой школе: слабый класс растёт заметно медленнее +- `dotnet test` проходит + +## Стоп + +Не вводить оценки. Не менять формулу зарплаты. Не трогать тепло и учебник. diff --git a/docs/phases/51-warmth-lesson.md b/docs/phases/51-warmth-lesson.md new file mode 100644 index 0000000..f203d71 --- /dev/null +++ b/docs/phases/51-warmth-lesson.md @@ -0,0 +1,37 @@ +# Фаза 51. Тепло на уроке + +## Зависимости + +- [Фаза 48](48-lesson-teacher-present.md) +- [Фаза 32](32-weather-warmth.md) + +## Зачем + +Замёрзший сидит на уроке так же, как голодный: полоска уже есть, рост её должен чувствовать. + +## Задачи + +- [ ] `Gain` умножает `NeedFactor(голод) × NeedFactor(тепло)` +- [ ] Нет нужды `Warmth` в каталоге — множитель тепла 1, голод как сейчас +- [ ] Лог `lesson-cold` один раз на предмет в сутки, если на уроке, который иначе учил, + тепло ниже `BehaviorDef.needThreshold`; `thingDef` — предмет +- [ ] `docs/protocol.md` — тип лога +- [ ] В [`inventory.md`](../design/inventory.md) убрать «тепло урок не штрафует»: стало рядом + с голодом + +## Тесты, без которых фаза не закрыта + +- [ ] Тепло 0.1 и голод 1 → меньший прирост, чем оба по 1 +- [ ] Тепло 1 и голод 0.1 → как нынешний голодный, тепло не «вылечивает» +- [ ] Нет `Warmth` в каталоге — голодный тест фазы 21 по-прежнему зелёный +- [ ] За урок на холоде одна строка лога, не по числу тиков +- [ ] Учителя нет в кабинете — лога холода нет (урок не учил) + +## Критерий готовности + +- Январь без техника: полоска тепла падает, навык за тот же урок растёт медленнее, чем в мае +- `dotnet test` проходит + +## Стоп + +Не болеть от промокания. Не штрафовать ходьбу. Не трогать учебник. diff --git a/docs/phases/52-textbook-lesson.md b/docs/phases/52-textbook-lesson.md new file mode 100644 index 0000000..136cbf5 --- /dev/null +++ b/docs/phases/52-textbook-lesson.md @@ -0,0 +1,38 @@ +# Фаза 52. Учебник на уроке + +## Зависимости + +- [Фаза 48](48-lesson-teacher-present.md) +- [Фаза 35](35-appropriateness.md) + +## Зачем + +Утренний `RepackBag` уже кладёт учебники сегодняшних предметов в сумку. Урок начинает это +проверять. + +## Задачи + +- [ ] Рост урока полный, только если в **сумке** есть экземпляр с `subject` этого предмета +- [ ] Шкафчик и дом не считаются: ученик в кабинете +- [ ] Нет учебника — множитель `BehaviorDef.lessonNoTextbookFactor`, ваниль 0.5 +- [ ] Валидатор: число 0–1 +- [ ] Лог `lesson-no-textbook` один раз на предмет в сутки; `thingDef` — предмет +- [ ] `docs/protocol.md` — тип лога + +## Тесты, без которых фаза не закрыта + +- [ ] Учебник математики в сумке → полный рост (при учителе на месте) +- [ ] Тот же учебник в шкафчике или дома → рост как с коэффициентом 0.5 +- [ ] Учебник другого предмета в сумке не закрывает этот урок +- [ ] Одна строка лога за сутки на предмет, не по числу тиков +- [ ] Пак без поля — 0.5, каталог не падает + +## Критерий готовности + +- Слабая переноска, учебник остался дома — на уроке растут хуже одноклассника с книгой +- `dotnet test` проходит + +## Стоп + +Не заставлять идти за учебником отдельной целью. Не замедлять ходьбу перегрузом. Не тратить +деньги на замену. diff --git a/docs/phases/53-weather-commute.md b/docs/phases/53-weather-commute.md new file mode 100644 index 0000000..5e35432 --- /dev/null +++ b/docs/phases/53-weather-commute.md @@ -0,0 +1,38 @@ +# Фаза 53. Погода на дороге + +## Зависимости + +- [Фаза 18](18-presence-walking.md) +- [Фаза 32](32-weather-warmth.md) + +## Зачем + +В январе опаздывают чаще, чем в мае. Черта уже двигает запас на дорогу; осадки — тоже. + +## Задачи + +- [ ] `BehaviorDef.commuteRainMinutes` и `commuteSnowMinutes`; ваниль 3 и 6 +- [ ] Валидатор: не отрицательные +- [ ] `DayPlans.Build` принимает уже посчитанные лишние минуты; `HSchool.Ai` не знает осадков +- [ ] Симуляция при сборке плана дня подставляет минуты из текущей улицы +- [ ] Лог на каждого не пишем — опоздание видно местом +- [ ] В [`inventory.md`](../design/inventory.md) поправка: погода влияет на дорогу, не только + на переодевание + +## Тесты, без которых фаза не закрыта + +- [ ] Снег → `AppearAt` раньше на `commuteSnowMinutes` относительно ясного того же дня + (тот же человек, сид, дата без осадков) +- [ ] Дождь → на `commuteRainMinutes` +- [ ] Ясно → минуты как сейчас, только черты и бросок +- [ ] Пак без полей — 0 лишних минут, каталог не падает +- [ ] Тот же сид и снег → тот же `AppearAt` + +## Критерий готовности + +- Снежным утром класс наполняется позже, чем ясным; ленивый по-прежнему ближе к звонку +- `dotnet test` проходит + +## Стоп + +Не мочить одежду. Не пропускать день из-за мороза. Не бампить кадр часов. diff --git a/docs/phases/README.md b/docs/phases/README.md index 14f4b69..f27cdc5 100644 --- a/docs/phases/README.md +++ b/docs/phases/README.md @@ -222,3 +222,18 @@ | [47. Пак romance](47-romance-pack.md) | ⬜ | Ориентация, симпатия, пара 18+, патч тем | 46 стоит на 37 и 42; 47 — на 42 и 23. `example` не трогать. + +## Вне очереди + +Дизайн: [`../design/off-queue.md`](../design/off-queue.md). + +Склейка уже живых систем, не новый кусок игры. Номера не спорят со срезом 10. После каждой +фазы школа целая. 50–52 стоят на 48; 53 — на 18 и 32, можно параллельно с 48. + +| Фаза | Статус | Зачем | +| --- | --- | --- | +| [48. Учитель в кабинете](48-lesson-teacher-present.md) | ✅ | Нет учителя на месте — нет роста навыка | +| [50. Навык учителя](50-teacher-skill-gain.md) | ⬜ | Соискатель 78 учит лучше, чем 41 | +| [51. Тепло на уроке](51-warmth-lesson.md) | ⬜ | Замёрзший учится хуже, как голодный | +| [52. Учебник на уроке](52-textbook-lesson.md) | ⬜ | Нет в сумке — половинный рост | +| [53. Погода на дороге](53-weather-commute.md) | ⬜ | Снег и дождь добавляют минуты к приходу | diff --git a/docs/protocol.md b/docs/protocol.md index 1df2394..1359a27 100644 --- a/docs/protocol.md +++ b/docs/protocol.md @@ -415,8 +415,9 @@ substring of the caption or type. `sort` is `time`. `dir` is `asc` or `desc` (de arrive in one response. Row `type` values: `action-started`, `action-ended`, `apparel-replaced` (morning issue, when -phase 34 appends it), `apparel-changed` (dressing, when phase 35 appends it). `thingDef` is -the action or apparel def the caption was built from. +phase 34 appends it), `apparel-changed` (dressing, when phase 35 appends it), +`lesson-no-teacher` (the assigned teacher was not standing in the lesson room). `thingDef` is +the action, apparel or subject def the caption was built from. ```json { diff --git a/src/HSchool.Server/mods/core/localizations/en.jsonc b/src/HSchool.Server/mods/core/localizations/en.jsonc index c587ff1..e6ece25 100644 --- a/src/HSchool.Server/mods/core/localizations/en.jsonc +++ b/src/HSchool.Server/mods/core/localizations/en.jsonc @@ -191,5 +191,6 @@ "ActionEnded": "finished: {0}", "ApparelReplaced": "got a new {0}", "ApparelChanged": "changed clothes: {0}", + "LessonNoTeacher": "lesson without a teacher: {0}", "core": "Core", } diff --git a/src/HSchool.Server/mods/core/localizations/ru.jsonc b/src/HSchool.Server/mods/core/localizations/ru.jsonc index affb2e6..28ce406 100644 --- a/src/HSchool.Server/mods/core/localizations/ru.jsonc +++ b/src/HSchool.Server/mods/core/localizations/ru.jsonc @@ -191,5 +191,6 @@ "ActionEnded": "закончил: {0}", "ApparelReplaced": "получил новую {0}", "ApparelChanged": "переоделся: {0}", + "LessonNoTeacher": "урок без учителя: {0}", "core": "Базовая игра", } diff --git a/src/HSchool.Simulation/LessonLearningSystem.cs b/src/HSchool.Simulation/LessonLearningSystem.cs index f1c1e88..d0f1daa 100644 --- a/src/HSchool.Simulation/LessonLearningSystem.cs +++ b/src/HSchool.Simulation/LessonLearningSystem.cs @@ -8,10 +8,14 @@ namespace HSchool.Simulation; /// /// Grows skills for people who are actually in the lesson: at the room, not walking, not off -/// doing something else. The formula lives in . +/// doing something else, and with the assigned teacher standing there. The formula lives in +/// . /// internal static class LessonLearningSystem { + private static readonly QueryDescription Places = + new QueryDescription().WithAll(); + private static readonly QueryDescription People = new QueryDescription().WithAll(); @@ -38,11 +42,19 @@ internal static class LessonLearningSystem var weekday = SchoolDay.WeekdayIndex(school.Clock.Time); var catalog = school.Catalog; var world = school.World; + var places = new Dictionary(StringComparer.Ordinal); + world.Query( + in Places, + (ref PersonIdentity identity, ref Presence presence) => + { + places[identity.Id] = presence; + }); + world.Query( in People, (ref PersonIdentity identity, ref PersonSkills skills, ref PersonTraits traits, ref PersonNeeds needs, ref PersonRoles roles, ref Presence presence, ref PersonActivity activity) => { - if (activity.IsActive || presence.NodeId is null || presence.Path.Length > 0 || presence.RemainingMinutes > 0) + if (activity.IsActive || !IsStanding(presence)) { return; } @@ -56,12 +68,19 @@ internal static class LessonLearningSystem var lesson = CurrentLesson(school, person, weekday, slot.Index); if (lesson is null + || presence.NodeId is null || !presence.NodeId.Equals(lesson.RoomId, StringComparison.Ordinal) || !catalog.Subjects.TryGetValue(lesson.Subject, out var subject)) { return; } + if (!TeacherStandingIn(places, lesson.TeacherId, lesson.RoomId)) + { + school.TryLogLessonOnce(personId, PersonLogTypes.LessonNoTeacher, lesson.Subject); + return; + } + var hunger = needs.Values.GetValueOrDefault("Hunger", 1f); foreach (var share in subject.Skills) { @@ -87,6 +106,17 @@ internal static class LessonLearningSystem }); } + private static bool IsStanding(Presence presence) => + presence.NodeId is not null && presence.Path.Length == 0 && presence.RemainingMinutes <= 0; + + private static bool TeacherStandingIn( + Dictionary places, + string teacherId, + string roomId) => + places.TryGetValue(teacherId, out var teacher) && IsStanding(teacher) + && teacher.NodeId is not null + && teacher.NodeId.Equals(roomId, StringComparison.Ordinal); + private static LessonPlacement? CurrentLesson(School school, Person person, int weekday, int period) { foreach (var lesson in Duty.LessonsToday(person, ClassOf(school, person), school.Timetable, weekday)) diff --git a/src/HSchool.Simulation/PersonLog.cs b/src/HSchool.Simulation/PersonLog.cs index 24534b1..8a4c75b 100644 --- a/src/HSchool.Simulation/PersonLog.cs +++ b/src/HSchool.Simulation/PersonLog.cs @@ -14,6 +14,7 @@ public static class PersonLogTypes public const string ActionEnded = "action-ended"; public const string ApparelReplaced = "apparel-replaced"; public const string ApparelChanged = "apparel-changed"; + public const string LessonNoTeacher = "lesson-no-teacher"; } /// @@ -58,6 +59,14 @@ public sealed record PersonLogEvent(string PersonId, DateTime Time, string Type, return string.Format(CultureInfo.InvariantCulture, catalog.Text(locale, key), name); } + if (Type.Equals(PersonLogTypes.LessonNoTeacher, StringComparison.Ordinal)) + { + var name = catalog.Subjects.TryGetValue(ThingDef, out var subject) + ? catalog.Label(locale, subject) + : ThingDef; + return string.Format(CultureInfo.InvariantCulture, catalog.Text(locale, "LessonNoTeacher"), name); + } + return Type; } } diff --git a/src/HSchool.Simulation/School.cs b/src/HSchool.Simulation/School.cs index bfcd068..679504f 100644 --- a/src/HSchool.Simulation/School.cs +++ b/src/HSchool.Simulation/School.cs @@ -17,6 +17,7 @@ public sealed class School : IDisposable private bool _disposed; private readonly List _dayLog = []; + private readonly HashSet _lessonLogOnce = new(StringComparer.Ordinal); internal School(int id, string name, DateTime startDate, DefCatalog? catalog, MapLayout? map) { @@ -135,10 +136,26 @@ public sealed class School : IDisposable { _dayLog.Clear(); LoggedActivity.Clear(); + _lessonLogOnce.Clear(); } internal void AppendDayLog(PersonLogEvent row) => _dayLog.Add(row); + /// + /// Lesson-quality rows fire every tick the condition holds. One key per person per day is enough. + /// + internal bool TryLogLessonOnce(string personId, string type, string subject) + { + var key = string.Concat(personId, "\0", type, "\0", subject); + if (!_lessonLogOnce.Add(key)) + { + return false; + } + + AppendDayLog(new PersonLogEvent(personId, Clock.Time, type, subject)); + return true; + } + public void QueueDecision(string personId) { ObjectDisposedException.ThrowIf(_disposed, this); diff --git a/tests/HSchool.Simulation.Tests/DecisionTests.cs b/tests/HSchool.Simulation.Tests/DecisionTests.cs index ffdb2e5..fcb967c 100644 --- a/tests/HSchool.Simulation.Tests/DecisionTests.cs +++ b/tests/HSchool.Simulation.Tests/DecisionTests.cs @@ -195,13 +195,18 @@ public class DecisionTests { var roster = RosterGenerator.Generate(catalog, map, seed, "Russia", TuesdayMorning); var pool = ApplicantPool.Create(catalog, roster, seed, "Russia", TuesdayMorning); + var hired = Staffing.Hire(catalog, map, roster, pool, pool.Applicants[0].Person.Id, Staffing.TeacherPosition, 1_000_000f); + Assert.Equal(StaffingError.None, hired.Error); + roster = hired.Roster; + pool = hired.Pool; + var teacherId = roster.People.First(person => person.IsStaff).Id; var schoolClass = roster.Classes.First(row => row.RoomId is "classroom-101" or "classroom-102" or "classroom-103" or "classroom-104"); var school = School.Create(seed, "Решения", TuesdayMorning, catalog, map); school.InstallPeople(roster, seed, "Russia", pool); school.SetTimetable(new Timetable( [ - new LessonPlacement(schoolClass.Id, "Mathematics", "t1", schoolClass.RoomId, Day: 1, Period: 1), + new LessonPlacement(schoolClass.Id, "Mathematics", teacherId, schoolClass.RoomId, Day: 1, Period: 1), new LessonPlacement(schoolClass.Id, "PhysicalEducation", "t2", "gym-hall", Day: 1, Period: 2), ], [])); diff --git a/tests/HSchool.Simulation.Tests/LessonTeacherPresentTests.cs b/tests/HSchool.Simulation.Tests/LessonTeacherPresentTests.cs new file mode 100644 index 0000000..e3fb020 --- /dev/null +++ b/tests/HSchool.Simulation.Tests/LessonTeacherPresentTests.cs @@ -0,0 +1,230 @@ +using Arch.Core; +using HSchool.Ai; +using HSchool.Content; +using HSchool.People; +using HSchool.Schedule; + +namespace HSchool.Simulation.Tests; + +public class LessonTeacherPresentTests +{ + private static readonly DateTime TuesdayMorning = new(2012, 4, 3, 6, 0, 0, DateTimeKind.Utc); + private static readonly DateTime LessonStart = new(2012, 4, 3, 8, 30, 0, DateTimeKind.Utc); + + [Fact] + public void PupilAndTeacherStanding_GainsSkill() + { + var (school, homeroom, pupilId, teacherId) = StaffedMath(); + using (school) + { + AdvanceTo(school, LessonStart); + SetPlace(school, pupilId, homeroom); + SetPlace(school, teacherId, homeroom); + var before = SkillOf(school, pupilId, "Mathematics"); + + LessonLearningSystem.Apply(school, 45); + + Assert.True(SkillOf(school, pupilId, "Mathematics") > before); + } + } + + [Fact] + public void TeacherInAnotherRoom_DoesNotGain() + { + var (school, homeroom, pupilId, teacherId) = StaffedMath(); + using (school) + { + AdvanceTo(school, LessonStart); + SetPlace(school, pupilId, homeroom); + SetPlace(school, teacherId, "restroom-1"); + var before = SkillOf(school, pupilId, "Mathematics"); + + LessonLearningSystem.Apply(school, 45); + + Assert.Equal(before, SkillOf(school, pupilId, "Mathematics")); + } + } + + [Fact] + public void TeacherWalking_DoesNotGain() + { + var (school, homeroom, pupilId, teacherId) = StaffedMath(); + using (school) + { + AdvanceTo(school, LessonStart); + SetPlace(school, pupilId, homeroom); + SetPlace(school, teacherId, "corridor-1", path: [homeroom], remaining: 1.5f); + var before = SkillOf(school, pupilId, "Mathematics"); + + LessonLearningSystem.Apply(school, 45); + + Assert.Equal(before, SkillOf(school, pupilId, "Mathematics")); + } + } + + [Fact] + public void MissingTeacherId_DoesNotGain() + { + var (school, homeroom, pupilId, _) = StaffedMath(teacherId: "nobody"); + using (school) + { + AdvanceTo(school, LessonStart); + SetPlace(school, pupilId, homeroom); + var before = SkillOf(school, pupilId, "Mathematics"); + + LessonLearningSystem.Apply(school, 45); + + Assert.Equal(before, SkillOf(school, pupilId, "Mathematics")); + } + } + + [Fact] + public void NoTeacher_LogsOncePerSubject() + { + var (school, homeroom, pupilId, teacherId) = StaffedMath(); + using (school) + { + AdvanceTo(school, LessonStart); + SetPlace(school, pupilId, homeroom); + SetPlace(school, teacherId, "restroom-1"); + + LessonLearningSystem.Apply(school, 5); + LessonLearningSystem.Apply(school, 5); + + var rows = school.DayLog + .Where(row => row.PersonId == pupilId && row.Type == PersonLogTypes.LessonNoTeacher) + .ToArray(); + Assert.Single(rows); + Assert.Equal("Mathematics", rows[0].ThingDef); + Assert.Equal( + "урок без учителя: Математика", + rows[0].Caption(school.Catalog!, "ru")); + } + } + + [Fact] + public void HungryStillGainsLess_WhenTeacherIsPresent() + { + var (school, homeroom, firstId, teacherId) = StaffedMath(); + using (school) + { + var schoolClass = school.Roster!.Classes.First(row => row.PupilIds.Contains(firstId)); + var secondId = schoolClass.PupilIds.First(id => id != firstId); + AdvanceTo(school, LessonStart); + SetPlace(school, firstId, homeroom); + SetPlace(school, secondId, homeroom); + SetPlace(school, teacherId, homeroom); + SetNeed(school, firstId, "Hunger", 0.2f); + SetNeed(school, secondId, "Hunger", 1f); + var hungryBefore = SkillOf(school, firstId, "Mathematics"); + var fullBefore = SkillOf(school, secondId, "Mathematics"); + + LessonLearningSystem.Apply(school, 45); + + var hungryGain = SkillOf(school, firstId, "Mathematics") - hungryBefore; + var fullGain = SkillOf(school, secondId, "Mathematics") - fullBefore; + Assert.True(fullGain > 0); + Assert.True(hungryGain > 0); + Assert.True(fullGain > hungryGain); + } + } + + private static (School School, string Homeroom, string PupilId, string TeacherId) StaffedMath( + string? teacherId = null) + { + var (catalog, map) = Vanilla(); + var roster = RosterGenerator.Generate(catalog, map, schoolSeed: 1, "Russia", TuesdayMorning); + var pool = ApplicantPool.Create(catalog, roster, schoolSeed: 1, "Russia", TuesdayMorning); + var hired = Staffing.Hire(catalog, map, roster, pool, pool.Applicants[0].Person.Id, Staffing.TeacherPosition, 1_000_000f); + Assert.Equal(StaffingError.None, hired.Error); + roster = hired.Roster; + pool = hired.Pool; + var hiredId = roster.People.First(person => person.IsStaff).Id; + var schoolClass = roster.Classes.First(row => + row.RoomId is "classroom-101" or "classroom-102" or "classroom-103" or "classroom-104"); + var school = School.Create(1, "Урок", TuesdayMorning, catalog, map); + school.InstallPeople(roster, seed: 1, "Russia", pool); + school.SetTimetable(new Timetable( + [new LessonPlacement(schoolClass.Id, "Mathematics", teacherId ?? hiredId, schoolClass.RoomId, Day: 1, Period: 1)], + [])); + school.ConfigurePresence(weekDays: 5, maxDecisionsPerTick: 10_000); + var pupil = schoolClass.PupilIds + .Select(id => school.Roster!.People.First(person => person.Id == id)) + .First(person => !person.Traits.Contains("Lazy")); + return (school, schoolClass.RoomId, pupil.Id, hiredId); + } + + private static void AdvanceTo(School school, DateTime until) + { + while (school.Clock.Time < until) + { + school.Tick(0.2d, 5d); + } + } + + private static void SetPlace(School school, string personId, string node, string[]? path = null, float remaining = 0f) + { + var query = new QueryDescription().WithAll(); + school.World.Query( + in query, + (ref PersonIdentity identity, ref Presence presence) => + { + if (identity.Id.Equals(personId, StringComparison.Ordinal)) + { + presence = new Presence(node, remaining, node, HeadingHome: false, path ?? []); + } + }); + } + + private static void SetNeed(School school, string personId, string need, float value) + { + var query = new QueryDescription().WithAll(); + school.World.Query( + in query, + (ref PersonIdentity identity, ref PersonNeeds needs) => + { + if (identity.Id.Equals(personId, StringComparison.Ordinal)) + { + needs.Values[need] = value; + } + }); + } + + private static float SkillOf(School school, string personId, string skill) + { + var value = float.NaN; + var query = new QueryDescription().WithAll(); + school.World.Query( + in query, + (ref PersonIdentity identity, ref PersonSkills skills) => + { + if (identity.Id.Equals(personId, StringComparison.Ordinal)) + { + value = skills.Values.GetValueOrDefault(skill); + } + }); + return value; + } + + private static (DefCatalog Catalog, MapLayout Map) Vanilla() + { + var root = Path.Combine(AppContext.BaseDirectory, "vanilla"); + var documents = new List(); + foreach (var path in Directory.EnumerateFiles(root, "*.*", SearchOption.AllDirectories)) + { + if (!path.EndsWith(".jsonc", StringComparison.OrdinalIgnoreCase) + && !path.EndsWith(".json", StringComparison.OrdinalIgnoreCase)) + { + continue; + } + + var relative = Path.GetRelativePath(root, path).Replace('\\', '/'); + documents.Add(new ContentDocument(CatalogLoader.CorePackId, relative, File.ReadAllText(path))); + } + + var catalog = new CatalogLoader().Load([CatalogLoader.CorePackId], documents); + var map = CatalogLoader.LastDefaultMap([CatalogLoader.CorePackId], documents); + Assert.NotNull(map); + return (catalog, map); + } +}