Фикс болтов- улетать из рук и левитировать после крафта не будут#273
Open
gordod3 wants to merge 6 commits into
Open
Фикс болтов- улетать из рук и левитировать после крафта не будут#273gordod3 wants to merge 6 commits into
gordod3 wants to merge 6 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
О PR
Болты, стрелы и т.п.
Projectiles(которые можно пере-использовать)с стали себя последнее время странно ввести - они выпадали из рук\контейнеров и были без гравитации при этом улетали в свободное плавание куда-то далеко пока не упирались во что-нибудь, подобрать их нельзя.Сложно сказать в чем именно заключается проблема ведь баг проявляется только при нагрузке сервера 50+ игроков, на локалке они работают нормально, но точно дело в том что они в первую очередь являются
ProjectileComponent, путем перебора\удаление компонентов они перестают летать если его удалить, углубившись понял что во всем виноватTileFrictionEventкоторый добавили, как я понял, чтобы исправить снаряды из корабельных орудий которые крутились вокруг своей оси если шаттл крутился во время стрельбы так что я решил что болты не корабельные снаряды и добавил проверку вProjectileComponentв частиOnTileFrictionчтобы она проверяла не является ли снаряд одноразовым.Хотя всё еще может быть проблема куда глубже, нужно понаблюдать как эти изменения сработают при нагруженном сервере, но уже как минимум все многоразовые снаряды перестали летать сами по себе на локалке и работают исправно.
Так же:
Я как понял в билде никогда не было логики скалирование визуального размера ЭМИ взрыва, или был, но его переделали так что вместе с Copilot я долго возился и читал каждую строчку в поиске места где впихнуть инициализацию размера визуального отображение(физически он корректно влияет на игру а визуально он всегда 5 клеток шириной), в итоге Copilot создал новый компонент для хранение значения радиуса
EmpBlastRangeComponentи прежде чем заспавнить взрыв, создаёт его предварительный объект, добавляет ему этот компонент, предварительно инициализируяrangeфизического взрыва а потом запускает спавн взрыва где уже там он берёт данные размера взрыва из компонента.Зачем / Баланс
Медиа
До фикса:
ForgeMonoBefore1.mp4
После:
ForgeMonoAfter1f.mp4
До фикса:
ForgeMonoBefore2.mp4
После:
ForgeMonoAfter2.mp4
Требования
Changelog
🆑 Gordod3