Погребок #12282
Погребок #12282
Conversation
Changelog status: ✔️ |
|
Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности: https://forum.taucetistation.org/t/fludilka-chast-2/28659/27824 |
|
Тогда нужно жесткое ограничение в количестве (Мб и есть, в коде не мастер), дабы не перенести по сотне каждого овоща на следующий раунд + работа ботаника в раунде |
в бочку помещается 200 реагента Добавлю в описание пра. |
|
Что будет с теми предметами что были в еде? |
хорошее уточнение, надо сделать так что бы можно было класть только пустые продукты |
Удалятся кнш. Оно просто переносит типы предметов что лежат в нём и в новом раунде создаёт с нуля предмет. Так и если в нём сыр а в сыре нож, то в следующем раунде на полке появится только сыр. Пустой. |
This reverts commit a52ec35.
|
разрешаю портировать sspersistence с тг |
|
неделя |
|
гусев, я тебя уже неделю в другом Пр-е жду #11329 если хотел сначала погреб доделать, то не надо было тот другой открывать |
|
Принимаю волевое решение, делаем вайтлист на вещи которые можно сохранять. Учитывая что он и так сделан, если кому-то понадобится более гибкая система - разрешаю ему переделать. |
|
Отлично, бот больше не ругается. Я всё. |
| @@ -0,0 +1,523 @@ | |||
| var/global/list/preservable_vegetables = list( | |||
There was a problem hiding this comment.
это полностью всё моё требование чтобы не сохранять непредсказуемые вещи которые унаследовались от родителя. я открыт к лучшим опциям.
There was a problem hiding this comment.
можно сделать флаги и генерировать списки при старте раунда, но вообще я не понимаю суть и почему именно реагент-контейнеры, в них что угодно может быть.
/snacks/grown/ содержит ссылку на семена, можно по ним смотреть. Обычные /snacks/ содержат реагенты.
There was a problem hiding this comment.
сморишь по /snacks/ и внезапно мыло можно сохранять между раундами, синдикатское к примеру.
There was a problem hiding this comment.
я понял, я просто о бочке думал и не въехал. Тогда реагентам флаги. Никто потом не вспомнит про то, что этот список надо поддерживать.
There was a problem hiding this comment.
Тогда реагентам флаги
У меня есть два объекта, один сохраняет только алкогольные напитки, второй - все напитки.
А флаг ОДИН и сгенерит он ОДИН список всех напитков.
Да и потом, с тем же успехом никто не вспомнит про флаг "МОЖЕТ_БЫТЬ_СОХРАНЕНО_В_БОЧКЕ" чтобы его поддерживать. А так - есть целый листик куда можно писать всякое.
There was a problem hiding this comment.
Также, у флагов проблема с наследованием. Установил флаг к чипсины, а потом оказалось что от чипсины наследуется синдимыло. Или установил флаг у помидора, а от помидора наследуется мегасуперОП блюспесс томато.
| ) | ||
|
|
||
| var/global/list/preservable_reagents = list( | ||
| "nutriment", |
There was a problem hiding this comment.
и в другом ПР-е тебе уже объясняли как генерировать списки, но ты всё успел забыть. Хардкодить имена реагентов - плохая идея
There was a problem hiding this comment.
это я потребовал спустя неделю ожиданий.
пожалуйста линкани ПР где можно почитать как надо
| /obj/effect/landmark/maintenance_barrel_spawn/atom_init(mapload) | ||
| ..() | ||
| global.maintenance_barrels_landmarks += loc | ||
| return INITIALIZE_HINT_QDEL |
There was a problem hiding this comment.
я думаю, будет лучше сделать обычный спавнер бочки, лендмарки предпочтительно оставить как персистентные пометки. И точно не надо свой список еще один создавать, они и так все в глобальном списке.
There was a problem hiding this comment.
хотя да, ты хочешь рандомно. Я только пока не понимаю, как ты потом рандомные бочки с персистентностью связываешь, но можно оставить лейблом и взять какой-нибудь pick_landmarked_location или свой написать
There was a problem hiding this comment.
Потому что у каждой бочки/структуры есть её айди. Айди выставляется мапперами и плюс у техтоннельной бочки есть техтоннельный айди. При записи контентсов бочки в файл оно записывает ассоциативном списком по айдишникам бочек/объектов.
| /datum/continuity_object/barrels/load(savefile/S) | ||
| spawn_maintenance_barrel() |
There was a problem hiding this comment.
тут еще будет много раундов ревью. Если ты вдруг решишь не заканчивать и переключишься на другие ПР-ы - скажи пожалуйста сразу, чтобы я мог не тратить время.
There was a problem hiding this comment.
Хз с чего ты решил что я решу не заканчивать ПР на погребок который лелею уже полгода, но как знаешь.
Я тут сижу по КД 24/7 в ПРе про погребок. И жду тебя, Волас. Я всегда жду тебя, Волас. Мы все здесь летаем, Волас.
|
так, датумы переместил, по остальным вопросам отписал, пойду в переводопр. |
|
Ждёт этих самых "много раундов ревью" |
|
Случайно, брух |
|
Какой дебил придумал делать кнопку "закрыть с комментом" рядом с кнопкой "коммент" |
| @@ -0,0 +1,150 @@ | |||
| /datum/continuity_object/barrels | |||
There was a problem hiding this comment.
теперь сделай так, чтобы датум отвечал только за хранение и сериализацию данных. Он абстрактный, он не должен сам дергать и искать объекты в игре (и тем более что-то спавнить). Это уже объекты в игре должны каким-то образом опрашивать соответствующий датум о возможно сохраненных для них данных, и забирать их из датума.
There was a problem hiding this comment.
чего бля, он ничего не спавнит, одумайся
There was a problem hiding this comment.
/datum/continuity_object/barrels/load(savefile/S)
spawn_maintenance_barrel()
я вижу
|
@volas а мне ведь нужен был этот ПР чтобы сделать два других continuity пр-а. Музей и табличка лучший работник месяца.... |
|
Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад. |
|
ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув |
|
Спасибо воласу за счастливое детство |
|
У тебя классные спрайты. Не знаю почему такие поездатые бочки так и не добавили. |
|
кодпроблемы. Я кстати не знаю, мне надо делать систему персистентности или уже пофигу? |
Мне пофигу. Я свой билд делаю)) А больше никому система не нужна была, так что... Меньше работы тебе) |
Описание изменений
Добавлены объекты что переносят еду и напитки между раундами.
Бочка - переносит напитки и позволяет крафтить алкоголь и неготовый сыр между раундами. Помещается 200 реагента.
Стол - переносит еду и позволяет донастаивать неготовый сыр между раундами. Помещается 4 предмета.
Корзина - переносит овощи и фрукты. Помещается 10 предметов.
Стоят они только на боксе и гамме, между картами предметы тоже переносятся, так что всё ок. На прометее и дельте некуда ставить, а на фальконе не нужно, имхо.
Почему и что этот ПР улучшит
СтРаТеГиЯ игры на поваре между раундами. Хороший тон положить овощей в корзину, да поставить готовиться сыр чтобы следующему повару было приятно :dovolen:
Авторство
AndreyGysev и все все все, кто помогал в дискорде.
Чеинжлог
🆑