Skip to content
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

Goal population settings to pick difficult goals based on players count #1248

Closed

Conversation

vakuor
Copy link

@vakuor vakuor commented Jun 29, 2024

Описание PR

Таска от Dexler'a.

Создание градации целей станции.
При высоком онлайне (>50) выпадают цели X
При низком онлайне (<50) выпадают цели Y

Добавил настройку для целей станции - параметр goalType в прототипе.
Значения могут быть AnyPopulation, LowPopulation, HighPopulation.
Пример из goals.yml

  type: stationGoal
  id: StationGoalGuests
  text: station-goal-guests
  goalType: AnyPopulation

Нужно, чтобы кто-то настроил эти параметры в goals.yml на каждую из целей. Без параметра в прототипе цель считается AnyPopulation.

Также число 50 записал константой в StationGoalPaperSystem.cs.
По-хорошему оно должно быть где-то в CVars.

private const int PlayersCountWhenPopulationIsHigh = 50;

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 vakuor

  • tweak: Изменен алгоритм выдачи цели станции! Теперь цели выдаются в зависимости от количества игроков. Мало игроков - проще цель. Много игроков - клоун ворвётся в ваш офис и забьёт вас едой.

@Kit0vras
Copy link

тебе стоит совместить это с space-syndicate/space-station-14#2338

/// <summary>
/// Idk how to make configs so let's put it here :\
/// </summary>
private const int PlayersCountWhenPopulationIsHigh = 50;
Copy link

Choose a reason for hiding this comment

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

перенести в CVar

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Jul 19, 2024
@Ady4ik
Copy link

Ady4ik commented Aug 21, 2024

@vakuor hello?

@stalengd stalengd closed this Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge Conflict Some content has conflicts.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants