Skip to content
This repository was archived by the owner on Apr 29, 2023. It is now read-only.

[WIP] Third Method of Rage AA Detector#182

Open
BlackYuzia wants to merge 4 commits intoBeepIsla:masterfrom
BlackYuzia:BlackYuzia-lbydelta-checker
Open

[WIP] Third Method of Rage AA Detector#182
BlackYuzia wants to merge 4 commits intoBeepIsla:masterfrom
BlackYuzia:BlackYuzia-lbydelta-checker

Conversation

@BlackYuzia
Copy link
Copy Markdown
Contributor

@BlackYuzia BlackYuzia commented Nov 14, 2020

don't punch me :d

Why did I create a PR with semi-finished (I mean dirty) code?

I want to get attention and finish the tests.
I made these changes a month ago. But so far no one has finished the tests.

General

So, this PR should fix AA detector.
In current method I was normalize angles (so this may work correctly)

Tasks:

  • Detect Rage AA
  • Complete tests
  • Complete a lot of more tests
  • Rename all LBYDeltaChecker words to AntiAim

About merging: DO NOT DO THAT WITHOUT 100% TESTS.

I was complete some tests. They say - cheaters would be detect by using rage AA in 8 cases of 10.
But we should complete more test. I want found did this method can have false positive or not and etc.

Tests tasks

  • Found did this method have false positive results
  • Found did this method can detect 100% of rage AA cheaters or not.
  • Found did tihs method can detect legit AA or not. (If not - I will try create method for Legit AA)

I was check 3-5 demos (don't remember how many actualy) with 1-5 rage cheaters per demo.
Some code in PR (like this or this etc should be removed or decrease)

Sync Master Branch with Original Repo
Third method (hope he is work correctly) for detect AA.
@BlackYuzia
Copy link
Copy Markdown
Contributor Author

Call me a Developer of methods 👀

@BlackYuzia
Copy link
Copy Markdown
Contributor Author

BlackYuzia commented Nov 14, 2020

Oh yep. This PR should fix #158 with rage AA detects issue.

Comment thread detectors/AntiAim.js
Comment on lines +120 to +129
const currentWeaponIsGrenade = () => {
const weapons = [
"weapon_hegrenade",
"weapon_flashbang",
"weapon_smokegrenade",
"weapon_decoy",
];

return weapons.includes(weapon);
};
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What about molotoves for T and CT? (weapon_molotov and weapon_incgrenade respectively)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I will add this, ok.
(Actualy, this is not critical)

Comment thread detectors/AntiAim.js
const m_nDuckTimeMsecs = this.parent.suspectPlayer.getProp("DT_Local", "m_nDuckTimeMsecs");
const m_nJumpTimeMsecs = this.parent.suspectPlayer.getProp("DT_Local", "m_nJumpTimeMsecs");
const m_flLastDuckTime = this.parent.suspectPlayer.getProp("DT_Local", "m_flLastDuckTime");
// const m_bIsMoving = null;//this.parent.suspectPlayer.getProp("DT_AI_BaseNPC", "m_bIsMoving"); // ! don't work
Copy link
Copy Markdown
Owner

@BeepIsla BeepIsla Nov 14, 2020

Choose a reason for hiding this comment

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

Likely because this is only for bots and not for humans? You can still check if a human is moving by just checking their velocity I guess

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I will try this, thx :3

@BeepIsla
Copy link
Copy Markdown
Owner

About merging: DO NOT DO THAT WITHOUT 100% TESTS.

I am waiting for demos.... #161 (comment)

@Padonsk
Copy link
Copy Markdown

Padonsk commented Feb 7, 2021

Pls add a threshold for an automatic verdict on AA: how much was AIM and how much WH was (for example* >=10 aim + >=5WH) .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants