Merge branch 'phase/48-lesson-teacher-present'
This commit is contained in:
Leonid Pershin
@@ -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
|
||||
|
||||
@@ -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`; лога нет |
|
||||
@@ -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` проходит
|
||||
|
||||
## Стоп
|
||||
|
||||
Не масштабировать рост навыком учителя. Не трогать тепло, учебник и дорогу. Не бампить
|
||||
протокол. Не писать лог учителю.
|
||||
@@ -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` проходит
|
||||
|
||||
## Стоп
|
||||
|
||||
Не вводить оценки. Не менять формулу зарплаты. Не трогать тепло и учебник.
|
||||
@@ -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` проходит
|
||||
|
||||
## Стоп
|
||||
|
||||
Не болеть от промокания. Не штрафовать ходьбу. Не трогать учебник.
|
||||
@@ -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` проходит
|
||||
|
||||
## Стоп
|
||||
|
||||
Не заставлять идти за учебником отдельной целью. Не замедлять ходьбу перегрузом. Не тратить
|
||||
деньги на замену.
|
||||
@@ -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` проходит
|
||||
|
||||
## Стоп
|
||||
|
||||
Не мочить одежду. Не пропускать день из-за мороза. Не бампить кадр часов.
|
||||
@@ -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) | ⬜ | Снег и дождь добавляют минуты к приходу |
|
||||
|
||||
+3
-2
@@ -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
|
||||
{
|
||||
|
||||
@@ -191,5 +191,6 @@
|
||||
"ActionEnded": "finished: {0}",
|
||||
"ApparelReplaced": "got a new {0}",
|
||||
"ApparelChanged": "changed clothes: {0}",
|
||||
"LessonNoTeacher": "lesson without a teacher: {0}",
|
||||
"core": "Core",
|
||||
}
|
||||
|
||||
@@ -191,5 +191,6 @@
|
||||
"ActionEnded": "закончил: {0}",
|
||||
"ApparelReplaced": "получил новую {0}",
|
||||
"ApparelChanged": "переоделся: {0}",
|
||||
"LessonNoTeacher": "урок без учителя: {0}",
|
||||
"core": "Базовая игра",
|
||||
}
|
||||
|
||||
@@ -8,10 +8,14 @@ namespace HSchool.Simulation;
|
||||
|
||||
/// <summary>
|
||||
/// Grows skills for people who are actually in the lesson: at the room, not walking, not off
|
||||
/// doing something else. The formula lives in <see cref="HSchool.Ai.LessonLearning"/>.
|
||||
/// doing something else, and with the assigned teacher standing there. The formula lives in
|
||||
/// <see cref="HSchool.Ai.LessonLearning"/>.
|
||||
/// </summary>
|
||||
internal static class LessonLearningSystem
|
||||
{
|
||||
private static readonly QueryDescription Places =
|
||||
new QueryDescription().WithAll<PersonIdentity, Presence>();
|
||||
|
||||
private static readonly QueryDescription People =
|
||||
new QueryDescription().WithAll<PersonIdentity, PersonSkills, PersonTraits, PersonNeeds, PersonRoles, Presence, PersonActivity>();
|
||||
|
||||
@@ -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<string, Presence>(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<string, Presence> 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))
|
||||
|
||||
@@ -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";
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -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;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -17,6 +17,7 @@ public sealed class School : IDisposable
|
||||
|
||||
private bool _disposed;
|
||||
private readonly List<PersonLogEvent> _dayLog = [];
|
||||
private readonly HashSet<string> _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);
|
||||
|
||||
/// <summary>
|
||||
/// Lesson-quality rows fire every tick the condition holds. One key per person per day is enough.
|
||||
/// </summary>
|
||||
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);
|
||||
|
||||
@@ -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),
|
||||
],
|
||||
[]));
|
||||
|
||||
@@ -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<PersonIdentity, Presence>();
|
||||
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<PersonIdentity, PersonNeeds>();
|
||||
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<PersonIdentity, PersonSkills>();
|
||||
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<ContentDocument>();
|
||||
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);
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user