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

Added spawnflags for keep player angles & velocity in trigger_teleport #747

Merged
merged 4 commits into from Apr 28, 2022
Merged

Added spawnflags for keep player angles & velocity in trigger_teleport #747

merged 4 commits into from Apr 28, 2022

Conversation

SmileyAG
Copy link
Contributor

@SmileyAG SmileyAG commented Apr 7, 2022

@afwn90cj93201nixr2e1re
Copy link
Contributor

afwn90cj93201nixr2e1re commented Apr 15, 2022

Чуть было не опубликовал комментарий без начальной точки. Это действительно один из тех самых PR которые стоило бы замерджить, но вроде бы сама реализация trigger_teleport подразумевает под собой установку специфичных углов при появлении на точке Б, не так ли? Просто игроки наверное с сохранением velocity вылетать начнут, но для DR серверов это действительно полезно, можно было бы уровни реализовывать и проходить карты не теряя скорость.

@SmileyAG
Copy link
Contributor Author

SmileyAG commented Apr 15, 2022

Чуть было не опубликовал комментарий без начальной точки. Это действительно один из тех самых PR которые стоило бы замерджить, но вроде бы сама реализация trigger_teleport подразумевает под собой установку специфичных углов при появлении на точке Б, не так ли? Просто игроки наверное с сохранением velocity вылетать начнут, но для DR серверов это действительно полезно, можно было бы уровни реализовывать и проходить карты не теряя скорость.

Это требует специфичных углов лишь в том случае, когда тебе надо двигаться в абсолютно другом направлении. Например, я двигался вперёд по направлению к Yaw 180 (X скорость: -700 юнитов), но после входа в телепорт, в зависимости от карты мне для продвижения надо теперь двигаться например по направлению к Yaw 90 (Y скорость: 700 юнитов). Поэтому в таких ситуациях приходится перенаправлять скорость в нужную ось за счёт установления особых углов в info_teleport_destination энтити и специального spawnflag'а для trigger_teleport.

Есть конечно же карты, где при выходе из телепорта ты всегда движешься в одном и том же направлении и поэтому его скорость не нужно перенаправлять, как например agtricks из Adrenaline Gamer мода, но я честно мало таких карт встречал пока в bspguy возился.

Я не думаю что мапперам нужно что-то кроме основных 4х углов (0, 90, 180, 270). Было бы странно видеть, если кто-то из мапперов захочет игрока двигаться по диагонали, лул.

@afwn90cj93201nixr2e1re
Copy link
Contributor

afwn90cj93201nixr2e1re commented Apr 15, 2022

но я честно мало таких карт встречал пока в bspguy возился

Ага, для ZM мода тоже много всяких 'разумных' людей телепорты подобным образом реализовывали от того игрокам зачастую даже не понятно где они находятся и они вертят прицелом во все стороны в надежде найти что-нибудь.

Насчет std:: функций не уверен только.
std::round, std::hypot, проверь, используются ли они вообще где-нибудь в коде, либо же есть их аналоги? Приведение к int'ам точно есть. Если есть, то скорее всего попросят заменить на существующие аналоги.

@SmileyAG
Copy link
Contributor Author

@afwn90cj93201nixr2e1re Вообщем, помог мне Juice сделать redirect скорости во все направления, теперь это не хардкод на 4 направления как это было до этого.

Насчет std:: функций не уверен только.

std::hypot заменил на Length2D из HLSDK.

@afwn90cj93201nixr2e1re
Copy link
Contributor

@s1lentq

Minor refactoring
@s1lentq s1lentq merged commit 21dab90 into s1lentq:master Apr 28, 2022
ShadowsAdi added a commit to ShadowsAdi/ReGameDLL_CS that referenced this pull request May 22, 2022
ShadowsAdi added a commit to ShadowsAdi/ReGameDLL_CS that referenced this pull request May 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants