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

Точность и спад урона #188

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

wCATw
Copy link
Collaborator

@wCATw wCATw commented Dec 19, 2024

Что я изменил

Я перенес систему точности, уклонения и спада урона с билда RMC-14.
Также я изменил для теста некоторые прототипы.

Убедитесь что вы проверили и согласны со следующим

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

TODO

Записать кредитс билда RMC и указать автора оригинала.
Изменить значения компонентов в прототипах.

Вот структура для прототипов:
*ЛОГИКА - логичесоке значение (true/false)
*ЧИСЛО - целочисленное значение
*ПЛАВ_ЧИСЛО - числовое значение с десятичной дробью

5 новых компонентов

на снаряды [Projectile]

  • Точность [STProjectileAccuracy]
    • параметр thresholds заполняется по шаблону ниже
    • параметр accuracy: ЧИСЛО
    • параметр ignoreFriendlyEvasion: ЛОГИКА
thresholds:
- range: ПЛАВ_ЧИСЛО
  falloff: ЧИСЛО
  • Спад урона [STProjectileDamageFalloff]
    • параметр thresholds заполняется по шаблону ниже
thresholds:
- range: ПЛАВ_ЧИСЛО
  falloff: ЧИСЛО
  ignoreModifiers: ЛОГИКА

на оружие [Weapon/Gun]

  • Точность [STWeaponAccuracy]
    • параметр accuracyMultiplier: ЧИСЛО
    • параметр accuracyMultiplierUnwielded: ЧИСЛО
  • Спад урона [STWeaponDamageFalloff]
    • параметр falloffMultiplier: ЧИСЛО

на существ [Species/Mobs]

  • Размер [STSize]
    • параметр size: значение из перечня Small, Humanoid, VerySmallMutant, SmallMutant, Mutant, Big, Immobile

JerryImMouse
JerryImMouse previously approved these changes Dec 19, 2024
Copy link
Collaborator

@JerryImMouse JerryImMouse left a comment

Choose a reason for hiding this comment

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

По стилю и чистоте ниче не скажу, ахуенно. А вот в логику лезть не буду, даже не просите

@wCATw wCATw force-pushed the kotovskiy/accuracy branch from 2e96dd2 to 2ef7a22 Compare December 19, 2024 19:07
@wCATw wCATw self-assigned this Dec 27, 2024
@Tornado-Technology Tornado-Technology added the enhancement New feature or request label Jan 13, 2025
@Tornado-Technology
Copy link
Collaborator

@JerryImMouse ало, у нас компоненты с ST начинаются.

@Tornado-Technology Tornado-Technology dismissed JerryImMouse’s stale review January 13, 2025 05:16

Некорректное

Copy link
Collaborator

@Tornado-Technology Tornado-Technology left a comment

Choose a reason for hiding this comment

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

Я пока не решусь сделать полное ревью, очень громоздкая система, где все ещё вшито много бесполезных или сломанных механик. По всем вопросам трогать меня в дискорде.

/// Conversion from 13: evasion
/// </summary>
[DataField, AutoNetworkedField]
public FixedPoint2 Evasion = 0;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Разъеби лопатой эту хуйню, тут нормально бы смотрелись флоаты


public enum EvasionModifiers : int
{
Rest = -15,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Чисто RMC хуйня у нас такого нет

public enum EvasionModifiers : int
{
Rest = -15,
Invisibility = 1000,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Кровососы выебут тебя в очко

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants