-
Notifications
You must be signed in to change notification settings - Fork 20
Disable xray in maps with friendly markers disabled #1574
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
Disable xray in maps with friendly markers disabled #1574
Conversation
|
For what it's worth, after cherry picking #1553 into this branch (to fix bots crashing DM mode), I tried setting Not sure if this fix requires that the map natively supports DM, or if DM mode should be enabled in a different way. I also tried bot-dm-xray-highlight-bug.mp4(Notice how I am on team NSF and the Jinrai players are not highlighted. In spectate mode they are all highlighted.) |
|
Also I'm not sure if we're keeping the PR changes small in scope, but CNEO_Player::GetFogObscuredRatio probably needs to be updated with a IsTeamplay() check such that enemies from the same team are not fully visible to each other: float CNEO_Player::GetFogObscuredRatio(CBaseEntity* target) const
{
/*---[snip]---*/
if (GetTeamNumber() == targetPlayer->GetTeamNumber())
{
// Teammates are always labeled with IFF markers
return 0.0f;
}(CNEOBotVision::IsAbleToSee seems fine already in this context but we might need to sanity check later that we don't forget to keep teammates not omniscient of each other.) |
|
master...AdamTadeusz:neo:306_dontGlowDMPlayers heres how you would solve this issue. I was gonna wait until the dm fix was in so it could be tested with bots. |
Would you prefer if I remove that half from this PR and you can submit that seperately |
|
Not handling dm here |
Description
Title
Toolchain
Linked Issues