Im-dex / xray-162

XRay engine 1.6.2 (S.T.A.L.K.E.R: Call of Pripyat) evolution
Apache License 2.0
47 stars 12 forks source link

Some physics bugs #21

Closed ghost closed 7 years ago

ghost commented 7 years ago

There is a small bug with physics currently. It is most prevalent when you shoot and kill a stalker and they drop their weapon. About a third of the time the weapon will wildly vibrate around, and sometimes go flying through the air. It's actually pretty funny to witness but it's still a bug :P

Im-dex commented 7 years ago

@Swartz27 Thanks, for the report. Is this issue present on x64 build only or reproduces on the x86?

ghost commented 7 years ago

@Im-dex I have only tested on x64, I don't know if it happens on x86.

ForserX commented 7 years ago

Кстати, Декс, давно NanoBot наткнулся на причину багов физики в сталкере. Вроде код, отвечающий за ракеты, емнип. Было обсуждение на АМК форуме, я бы поискал пост, но зарёкся там больше не появляться.

Im-dex commented 7 years ago

@ForserX Главная причина багов физики - это в первую очередь убогий физ движок :) А за подсказку про амк форум спасибо, попробую поискать.

ForserX commented 7 years ago

Слушай, а у тебя бага с граф-поинтами не было на оригинальных локациях?

Im-dex commented 7 years ago

@ForserX Какого именно? Пока вроде ничего не замечал, но вполне мог и проглядеть.

ForserX commented 7 years ago

@Im-dex, у меня просто оригинальные локации на Mixed проверках вылетают.

Im-dex commented 7 years ago

Я только релиз собираю, поэтому не замечал

ForserX commented 7 years ago

Хм... Ну, моё железо не позволят для тестов в релизе компилить.

Im-dex commented 7 years ago

А в чем проблема скомпилить в релизе? Он вроде не особо отличается по прожорливости от дебага

ForserX commented 7 years ago

Оптимизация кода занимает час на моём ПК.

ForserX commented 7 years ago

@Im-dex А как быть с get_signature на релизе? https://pp.userapi.com/c837438/v837438782/4b0a4/im3H6RKWF_A.jpg

Im-dex commented 7 years ago

@ForserX А что там? И в выделенном куске вообще-то дебаг

ForserX commented 7 years ago

@Im-dex в дебаг я уже перевёл, чтоб собиралось. В оригинале он просто отсуствует на релиз-сборке.

ghost commented 7 years ago

Just got a crash that I hope helps in some way:

FATAL ERROR

[error]Expression : assertion failed [error]Function : CPHActivationShape::Create [error]File : X:\xray-162-master\code\engine\xrPhysics\PHActivationShape.cpp [error]Line : 177 [error]Description : _valid( start_pos )

Im-dex commented 7 years ago

@Swartz27 Could you please provide a crash dump for this case? It will be more helpful than a stacktrace.

ghost commented 7 years ago

@Im-dex Shit, I deleted my logs folder :( I'll try and get it to happen again, though it was very random.

ghost commented 7 years ago

I totally forgot about Nick Galko. He had an x64 repo, it doesn't run properly, but the one thing that did work right was the physics, and in fact he told me this commit has a fix for the physics bugs:

https://bitbucket.org/nick_galko/cop/commits/6295d37

Sorry for not relaying this sooner.

Im-dex commented 7 years ago

Nick Galko's collider fix does not work for me. I will try to rewrite this piece of shit at some day.

ghost commented 7 years ago

Can confirm this fixes it: https://github.com/ForserX/FRay-Project/commit/059f64eeb3cdecbab6c129067ac41e07a0a93d6e

Tested for more than an hour, physics bug is gone now.

ForserX commented 7 years ago

Угу. Мне кажется, что лучше во всех местах поменять dInfinity, т.к. под x64 они вычисляются не правильно.

Im-dex commented 7 years ago

@ForserX Само значение dInfinity вычисляется идентично для обеих архитектур. Но видимо из-за игрищ с пресижином и округлениями, которые нельзя полностью повторить для x64, и появляется данная проблема.