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

Minor improvements to bot code #15

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Minor improvements to bot code #15

wants to merge 3 commits into from

Conversation

rXn4
Copy link

@rXn4 rXn4 commented Feb 12, 2015

Changed some things that prevented me from having fun in bot matches.
Hope you dont mind that I make quite a lot of pull requests, just have a look at it.

@ruler501
Copy link
Contributor

I think this is good to be merged, I'll see what Victor thinks, then we can merge it.

@theonlypwner
Copy link
Member

  1. Aiming at the head
    Bots aim at the neck with automatic weapons, which is probably more accurate than aiming at the head (of moving targets). Aiming at the head with sniper rifles makes sense only when they are zoomed though.
  2. Lower aim offset for "good" bots
    Is this really necessary though?
  3. Less angle offset before shooting
    Just like the comment says, it should really "be skill based", so this change just increases the skill of all bots.
  4. Don't reset bots when they run into a wall
    This is a good change, so that they can try doing the stuff in the code below that section (turning and randomly jumping) before being reset.
  5. Throttling weapon changes
    Weapon changes are already throttled by m_iChangeWeaponDelay. Adding a global variable to throttle changes is a bad idea because it means that the maximum weapon change frequency is proportional to the number of clients connected to the server divided by the number of bots. Weapon change intervals should be for each individual bot. Having more human players or less bots should not affect how often bots change weapons.

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

3 participants