Skip to content

Новая рыбалка на основе механики Губов#123

Open
tag168043-dev wants to merge 4 commits intoShegare:masterfrom
tag168043-dev:feature
Open

Новая рыбалка на основе механики Губов#123
tag168043-dev wants to merge 4 commits intoShegare:masterfrom
tag168043-dev:feature

Conversation

@tag168043-dev
Copy link

@tag168043-dev tag168043-dev commented Feb 26, 2026 •
edited
Loading

Описание PR

Изменения в механике рыбалки

Почему / Баланс

Для веселья, увеличение популярности среди игрокоов

Технические детали

Добавления множества под-уровней лута для разных мест рыбалки

Медиа

Требования

  • Я добавил медиафайлы (скрины, видео) к этому PR или он не требует демонстрации в игре.
  • Я проверил что мои изменения не вызывают конфликтов и багов в сборке.
  • Я уверен, что мои изменения не нарушают лицензионные соглашения.

Список изменений

LuciferEOS
LuciferEOS reviewed Feb 27, 2026
View reviewed changes
Copy link

@LuciferEOS LuciferEOS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Чатгпт скажи мне как работают weights в yaml

Resources/Prototypes/_SIS/Entities/Objects/Specific/Fishing/loot_tables.yml
- id: PowerCellHighPrinted
- id: MobCatCake
- id: ClothingMaskWeldingGas
- id: AdvMopItem
Copy link

@LuciferEOS LuciferEOS Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не жирно ли на ряду с изолями?

Copy link

@LuciferEOS LuciferEOS Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И кортом тоже*

Copy link
Author

@tag168043-dev tag168043-dev Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Удачи их еще выловить, с кортом да вышла ошибочка но большинство предметов выбрано специально чтобы иметь какую-то ценность но не давать огромного преимущества, почти все можно изучить в рнд намного быстрее или же просто купить.

Resources/Prototypes/_SIS/Entities/Objects/Specific/Fishing/loot_tables.yml
children:
- id: MobMonkey
- id: MobGoose
- id: MobCorgi
Copy link

@LuciferEOS LuciferEOS Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Каво, выловил корги из унитаза?

Copy link
Author

@tag168043-dev tag168043-dev Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Карго корги миф?

Shegare
Shegare requested changes Feb 27, 2026
View reviewed changes
Copy link
Owner

@Shegare Shegare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Если цель была переопределить лут, то лучше в папке Губов удалить те же таблицы сущностей

Resources/Prototypes/_SIS/Entities/Objects/Specific/Fishing/loot_tables.yml Outdated
- id: MobBreadDog

- type: entityTable
id: WaterFishingLootTable
Copy link
Owner

@Shegare Shegare Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Такой id существует. id должны быть уникальными

Resources/Prototypes/_SIS/Entities/Objects/Specific/Fishing/loot_tables.yml Outdated
weight: 0.05

- type: entityTable
id: LavaFishingLootTable
Copy link
Owner

@Shegare Shegare Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Такой тоже

Resources/Prototypes/_SIS/Entities/Objects/Specific/Fishing/loot_tables.yml Outdated
weight: 0.1

- type: entityTable
id: PlasmaFishingLootTable
Copy link
Owner

@Shegare Shegare Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Такой тоже

Resources/Prototypes/_SIS/Entities/Objects/Specific/Fishing/loot_tables.yml Outdated
weight: 0.1

- type: entityTable
id: ToiletFishingLootTable
Copy link
Owner

@Shegare Shegare Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Такой тоже

Copy link
Author

@tag168043-dev tag168043-dev Feb 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

То-есть весь прокол сейчас в том что EntityTable с губов повторяется? в остальном все работает?

Copy link
Owner

@Shegare Shegare Feb 27, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

То-есть весь прокол сейчас в том что EntityTable с губов повторяется?

Да. id: должны быть уникальными. Если не хочешь везде заменять название id, а использовать уже существующие, то объявление старых лучше удалить

в остальном все работает?

Работать то оно наверное будет, просто это непредсказуемо и плохо читаемо

@Shegare
Copy link
Owner

Shegare commented Feb 27, 2026

изображение

Также сам ПР так себе оформлен:

  • Название ПРа должно быть на русском (по анологии со всеми остальными, так быстрее понять что было изменено, плюс "Добавлен прототип.умл" плохо описывает изменения, ну и приятнее когда всё в одном стиле)
  • Список изменений - это то что я копирую и вставляю в ченджлог. Если изменение отражается на игровой процесс то это место обязано быть заполненно, это не опция (опять же можно было посмотреть на примере других ПРок)

@tag168043-dev tag168043-dev changed the title Create loot_tables.yml Новая рыбалка на основе механики Губов Feb 28, 2026
@Samur7
Copy link
Contributor

Samur7 commented Mar 8, 2026

Предлагаю сделать рыбалку только для спонсоров 😎

@Samur7
Copy link
Contributor

Samur7 commented Mar 13, 2026

Сделай пока что драфтом

@Shegare
Copy link
Owner

Shegare commented Mar 13, 2026

Сделай пока что драфтом

Он скилл ишу, я сам доделаю, после того как с локализацией покончу

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Shegare Shegare Shegare requested changes

+2 more reviewers

@LuciferEOS LuciferEOS LuciferEOS left review comments

@Samur7 Samur7 Samur7 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Changes: yml S: Needs Review S: Untriaged size/M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tag168043-dev @Shegare @Samur7 @LuciferEOS