-
-
Notifications
You must be signed in to change notification settings - Fork 422
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Погребок #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 с тг |
Отлично, бот больше не ругается. Я всё. |
@@ -0,0 +1,523 @@ | |||
var/global/list/preservable_vegetables = list( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
весь этот файл - какой-то пипец
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
это полностью всё моё требование чтобы не сохранять непредсказуемые вещи которые унаследовались от родителя. я открыт к лучшим опциям.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
можно сделать флаги и генерировать списки при старте раунда, но вообще я не понимаю суть и почему именно реагент-контейнеры, в них что угодно может быть.
/snacks/grown/
содержит ссылку на семена, можно по ним смотреть. Обычные /snacks/
содержат реагенты.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
сморишь по /snacks/ и внезапно мыло можно сохранять между раундами, синдикатское к примеру.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
я понял, я просто о бочке думал и не въехал. Тогда реагентам флаги. Никто потом не вспомнит про то, что этот список надо поддерживать.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Тогда реагентам флаги
У меня есть два объекта, один сохраняет только алкогольные напитки, второй - все напитки.
А флаг ОДИН и сгенерит он ОДИН список всех напитков.
Да и потом, с тем же успехом никто не вспомнит про флаг "МОЖЕТ_БЫТЬ_СОХРАНЕНО_В_БОЧКЕ" чтобы его поддерживать. А так - есть целый листик куда можно писать всякое.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Также, у флагов проблема с наследованием. Установил флаг к чипсины, а потом оказалось что от чипсины наследуется синдимыло. Или установил флаг у помидора, а от помидора наследуется мегасуперОП блюспесс томато.
) | ||
|
||
var/global/list/preservable_reagents = list( | ||
"nutriment", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
и в другом ПР-е тебе уже объясняли как генерировать списки, но ты всё успел забыть. Хардкодить имена реагентов - плохая идея
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
это я потребовал спустя неделю ожиданий.
пожалуйста линкани ПР где можно почитать как надо
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
я думаю, будет лучше сделать обычный спавнер бочки, лендмарки предпочтительно оставить как персистентные пометки. И точно не надо свой список еще один создавать, они и так все в глобальном списке.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
хотя да, ты хочешь рандомно. Я только пока не понимаю, как ты потом рандомные бочки с персистентностью связываешь, но можно оставить лейблом и взять какой-нибудь pick_landmarked_location или свой написать
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Потому что у каждой бочки/структуры есть её айди. Айди выставляется мапперами и плюс у техтоннельной бочки есть техтоннельный айди. При записи контентсов бочки в файл оно записывает ассоциативном списком по айдишникам бочек/объектов.
/datum/continuity_object/barrels/load(savefile/S) | ||
spawn_maintenance_barrel() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
тут еще будет много раундов ревью. Если ты вдруг решишь не заканчивать и переключишься на другие ПР-ы - скажи пожалуйста сразу, чтобы я мог не тратить время.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хз с чего ты решил что я решу не заканчивать ПР на погребок который лелею уже полгода, но как знаешь.
Я тут сижу по КД 24/7 в ПРе про погребок. И жду тебя, Волас. Я всегда жду тебя, Волас. Мы все здесь летаем, Волас.
так, датумы переместил, по остальным вопросам отписал, пойду в переводопр. |
Ждёт этих самых "много раундов ревью" |
Случайно, брух |
Какой дебил придумал делать кнопку "закрыть с комментом" рядом с кнопкой "коммент" |
@@ -0,0 +1,150 @@ | |||
/datum/continuity_object/barrels |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
теперь сделай так, чтобы датум отвечал только за хранение и сериализацию данных. Он абстрактный, он не должен сам дергать и искать объекты в игре (и тем более что-то спавнить). Это уже объекты в игре должны каким-то образом опрашивать соответствующий датум о возможно сохраненных для них данных, и забирать их из датума.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
чего бля, он ничего не спавнит, одумайся
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/datum/continuity_object/barrels/load(savefile/S)
spawn_maintenance_barrel()
я вижу
@volas а мне ведь нужен был этот ПР чтобы сделать два других continuity пр-а. Музей и табличка лучший работник месяца.... |
Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад. |
ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув |
Спасибо воласу за счастливое детство |
У тебя классные спрайты. Не знаю почему такие поездатые бочки так и не добавили. |
кодпроблемы. Я кстати не знаю, мне надо делать систему персистентности или уже пофигу? |
Мне пофигу. Я свой билд делаю)) А больше никому система не нужна была, так что... Меньше работы тебе) |
Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности: https://forum.taucetistation.org/t/fludilka-chast-2/28659/37713 |
Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности: https://forum.taucetistation.org/t/loupop-i-reshenie-problemy-resov/44068/44 |
А могли бы уже давно в мыле кататься. |
Иногда я прихожу сюда чтобы снова посмотреть на то что же пошло не так... Ля будет забавно если в начале 2026-го года Волас закодит персистентность, я посмотрю что там он сделает и пойму что моему механу погребка оно вообще не соответствует и мне придётся переписывать новый механ персистентности чтобы добавить погребок, и всё по новой, ахах... хах... дааа |
Ну хоть спрайты апрувнуты. |
Описание изменений
Добавлены объекты что переносят еду и напитки между раундами.
Бочка - переносит напитки и позволяет крафтить алкоголь и неготовый сыр между раундами. Помещается 200 реагента.
Стол - переносит еду и позволяет донастаивать неготовый сыр между раундами. Помещается 4 предмета.
Корзина - переносит овощи и фрукты. Помещается 10 предметов.
Стоят они только на боксе и гамме, между картами предметы тоже переносятся, так что всё ок. На прометее и дельте некуда ставить, а на фальконе не нужно, имхо.
![image](https://private-user-images.githubusercontent.com/21008918/276152151-4c364ba6-91df-45a7-b3c8-6a9aacf2f5c6.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkzNjc0MDksIm5iZiI6MTczOTM2NzEwOSwicGF0aCI6Ii8yMTAwODkxOC8yNzYxNTIxNTEtNGMzNjRiYTYtOTFkZi00NWE3LWIzYzgtNmE5YWFjZjJmNWM2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjEyVDEzMzE0OVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWEwNjA3MWMxYWEwYzNlZTc1ZGVkZGRhZDQ1ZWZiZjcwOWE5MmQ1NjgzMzEwZDhlOTU2NTgyZTVhOWQwZDNiNzkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.qPBNMhGWPTHuD_mhK6p1ufhKRlItfdp7quIS9ZZONkc)
Почему и что этот ПР улучшит
СтРаТеГиЯ игры на поваре между раундами. Хороший тон положить овощей в корзину, да поставить готовиться сыр чтобы следующему повару было приятно :dovolen:
Авторство
AndreyGysev и все все все, кто помогал в дискорде.
Чеинжлог
🆑