Anonymous | Login | Signup for a new account | 2021-03-06 11:47 UTC | ![]() |
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
0003769 | Zandronum | [All Projects] Bug | public | 2020-03-18 23:48 | 2020-03-29 19:19 | ||||||||
Reporter | sosleepy | ||||||||||||
Assigned To | |||||||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||||||
Status | needs testing | Resolution | open | ||||||||||
Platform | OS | Ubuntu x86_64 | OS Version | Eoan | |||||||||
Product Version | 3.0 | ||||||||||||
Target Version | Fixed in Version | ||||||||||||
Summary | 0003769: Bots tries to jump to reach item when sv_nojump is true | ||||||||||||
Description | WAD attached for easy reproduction of the issue. My guess is the issue can be easily solved by editing the ternary at line 464 in src/botpath.cpp to check the value of sv_nojump. I would appreciate if someone taught me how to have the current values of the server variables to try it myself. | ||||||||||||
Attached Files | ![]() | ||||||||||||
![]() |
|
sosleepy (reporter) 2020-03-24 22:10 |
Fixed, needs testing since this fix is fairly invasive. https://bitbucket.org/Torr_Samaho/zandronum/pull-requests/225/player-and-bots-shouldnt-attempt-to-jump/diff [^] |
sosleepy (reporter) 2020-03-25 01:12 |
Considering the locations this method is used, I would say it is a safe change. $ grep -R CalcJumpHeight src/botpath.cpp: fixed_t jumpheight = ( pActor->IsKindOf (RUNTIME_CLASS (APlayerPawn)) ) ? static_cast<APlayerPawn*>( pActor )->CalcJumpHeight( ) : 60; src/d_player.h: fixed_t CalcJumpHeight( bool bAddStep = true ); src/p_user.cpp:fixed_t APlayerPawn::CalcJumpHeight( bool bAddStepZ ) |
sosleepy (reporter) 2020-03-25 19:00 |
There might be a better way, from these notes: https://zandronum.com/tracker/view.php?id=3775#c21236 [^] https://zandronum.com/tracker/view.php?id=3775#c21237 [^] The following values are relevant: BOTCMD_JUMP (59), BOTCMD_BEGINJUMPING (60), BOTCMD_STOPJUMPING (61). |
Torr Samaho (administrator) 2020-03-29 19:17 |
I added your patch. |
Only registered users can voice their support. Click here to register, or here to log in. | |
Supporters: | No one explicitly supports this issue yet. |
Opponents: | No one explicitly opposes this issue yet. |
![]() |
|||
Date Modified | Username | Field | Change |
2020-03-18 23:48 | sosleepy | New Issue | |
2020-03-18 23:48 | sosleepy | File Added: testbot.wad | |
2020-03-24 22:10 | sosleepy | Note Added: 0021232 | |
2020-03-25 01:12 | sosleepy | Note Added: 0021234 | |
2020-03-25 19:00 | sosleepy | Note Added: 0021238 | |
2020-03-29 19:17 | Torr Samaho | Note Added: 0021244 | |
2020-03-29 19:18 | Torr Samaho | Assigned To | => Torr Samaho |
2020-03-29 19:18 | Torr Samaho | Status | new => needs testing |
2020-03-29 19:19 | Torr Samaho | Assigned To | Torr Samaho => |
Copyright © 2000 - 2021 MantisBT Team |