Zandronum Chat on our Discord Server Get the latest version: 3.0
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0003769Zandronum[All Projects] Bugpublic2020-03-18 23:482020-03-29 19:19
Reportersosleepy 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Statusneeds testingResolutionopen 
PlatformOSUbuntu x86_64OS VersionEoan
Product Version3.0 
Target VersionFixed in Version 
Summary0003769: Bots tries to jump to reach item when sv_nojump is true
DescriptionWAD 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? file icon testbot.wad [^] (2,716 bytes) 2020-03-18 23:48

- Relationships

-  Notes
User avatar (0021232)
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 [^]
User avatar (0021234)
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 )
User avatar (0021238)
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).
User avatar (0021244)
Torr Samaho (administrator)
2020-03-29 19:17

 I added your patch.

Issue Community Support
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.

- Issue History
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 =>




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2021 MantisBT Team
Powered by Mantis Bugtracker