[11882]Fix rare freeze in MaNGOS::NearUsedPosDo - Ru-MaNGOS

rsa · 18.01.2012, 20:32

subj
https://github.com/mangosR2/mangos/c...da47d84502ced4
фриз редкий и проявляется на большой нагрузке, возможно только с mmap. но возможен и на обычном мангосе (причина - кривые вычисления с float точностью, иногда дают angle порядка 10**30 вместо 0)

PS не пора еще лавочку закрывать? тускло как-то...

**virusav** · 18.01.2012, 20:44

Не пора, год только начался.

rsa · 18.01.2012, 20:48

Цитата:

Сообщение от virusav

Не пора, год только начался.

через месяц вспомним, или через полгодика?

PS я не злопамятный. просто злой, и память хорошая (с).

**schmoozerd** · 24.01.2012, 00:51

With some changes in 11882. Thank you!

Except I miss something due to too much drinking, the only possible place for problems could have been i_angle, which I corrected to be normalized.

If you don't agree, please tell in which combinations you see a possible fail

rsa · 24.01.2012, 09:50

your normaization method in NearUsedPosDo more correct then my, but i think, normalization in GetAngle() still needed for other purposes (atan2 may returns broken value in some cases).

PS to zergtmn - хоть его-то не обвиняйте в рекламе R2, ладно?

**schmoozerd** · 24.01.2012, 13:41

according to http://www.cplusplus.com/reference/c...y/cmath/atan2/
atan2 returns a value in -Pi..Pi, hence the check done currently in GetAngle must return a value in 0..2Pi

rsa · 24.01.2012, 13:52

problem not in atan2, but in casting from long double to float. 0.0f may be interpreted as float MAX in some (very rare) cases.

18.01.2012, 20:32	#1
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	[11882]Fix rare freeze in MaNGOS::NearUsedPosDo subj https://github.com/mangosR2/mangos/c...da47d84502ced4 фриз редкий и проявляется на большой нагрузке, возможно только с mmap. но возможен и на обычном мангосе (причина - кривые вычисления с float точностью, иногда дают angle порядка 10**30 вместо 0) PS не пора еще лавочку закрывать? тускло как-то...

24.01.2012, 00:51	#4
schmoozerd MaNGOS Dev Регистрация: 17.11.2011 Сообщений: 99 Сказал(а) спасибо: 35 Поблагодарили 80 раз(а) в 26 сообщениях	With some changes in 11882. Thank you! Except I miss something due to too much drinking, the only possible place for problems could have been i_angle, which I corrected to be normalized. If you don't agree, please tell in which combinations you see a possible fail Последний раз редактировалось schmoozerd; 24.01.2012 в 00:53.

24.01.2012, 09:50	#5
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	your normaization method in NearUsedPosDo more correct then my, but i think, normalization in GetAngle() still needed for other purposes (atan2 may returns broken value in some cases). PS to zergtmn - хоть его-то не обвиняйте в рекламе R2, ладно? Последний раз редактировалось rsa; 24.01.2012 в 10:04.

Похожие темы
Тема	Автор	Раздел	Ответов	Последнее сообщение
[11658] Revert "[11298] NearUsedPosDo searcher should add only units with correct coordinates. Possi	newsbot	CMaNGOS Commits	0	20.06.2011 22:21
[11330] Use ObjectGuid to store Totem guids instead of uint64 and fix related rare crash.	newsbot	CMaNGOS Commits	0	11.04.2011 23:01
[11298] NearUsedPosDo searcher should add only units with correct coordinates. Possibly can help wit	newsbot	CMaNGOS Commits	0	30.03.2011 09:40
[9588] Really fix freeze :/	newsbot	CMaNGOS Commits	0	15.03.2010 16:30
[9587] Fixed freeze after recent loot roll code changes.	newsbot	CMaNGOS Commits	0	15.03.2010 16:30

18.01.2012, 20:44	#2
virusav Администратор Регистрация: 19.02.2010 Сообщений: 492 Сказал(а) спасибо: 55 Поблагодарили 341 раз(а) в 154 сообщениях	Не пора, год только начался.

24.01.2012, 13:41	#6
schmoozerd MaNGOS Dev Регистрация: 17.11.2011 Сообщений: 99 Сказал(а) спасибо: 35 Поблагодарили 80 раз(а) в 26 сообщениях	according to http://www.cplusplus.com/reference/c...y/cmath/atan2/ atan2 returns a value in -Pi..Pi, hence the check done currently in GetAngle must return a value in 0..2Pi

24.01.2012, 13:52	#7
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	problem not in atan2, but in casting from long double to float. 0.0f may be interpreted as float MAX in some (very rare) cases.