Skip to content

Conversation

@sunzenshen
Copy link
Contributor

@sunzenshen sunzenshen commented Sep 14, 2025

Description

Check for friendlies between bot and target before opening fire.

Toolchain

  • Windows MSVC VS2022

Linked Issues

@sunzenshen sunzenshen requested a review from a team September 14, 2025 21:38
@sunzenshen sunzenshen marked this pull request as draft September 23, 2025 04:03
@sunzenshen sunzenshen removed the request for review from a team September 28, 2025 06:23
@sunzenshen sunzenshen force-pushed the bot-reduce-friendly-fire branch from b331bc0 to d82d13f Compare October 12, 2025 04:46
@sunzenshen sunzenshen force-pushed the bot-reduce-friendly-fire branch from d82d13f to 7151e48 Compare October 22, 2025 04:36
@sunzenshen sunzenshen marked this pull request as ready for review October 22, 2025 04:56
@sunzenshen sunzenshen requested a review from a team October 22, 2025 04:56
@sunzenshen sunzenshen added the Enhancement New feature or request label Oct 25, 2025
Copy link
Contributor

@AdamTadeusz AdamTadeusz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if memory serves CTraceFilterChain requires that an entity is hit by both filters that make up the filter chain, so a custom CTraceFilterChain would have to be added where the shouldhit checks whether either of the filters that make it up would hit instead of only if both should hit, so im just gonna approve this for now, maybe add a TODO to simplify this in the future

@sunzenshen sunzenshen requested a review from a team November 7, 2025 10:55
@sunzenshen sunzenshen merged commit fbe3e92 into NeotokyoRebuild:master Nov 10, 2025
7 checks passed
@DESTROYGIRL DESTROYGIRL added the Bots Related to bot players label Dec 9, 2025
@sunzenshen sunzenshen deleted the bot-reduce-friendly-fire branch December 16, 2025 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bots Related to bot players Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants