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

Blood: Add 'pitchfork only' autoaim option #884

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

tmyqlfpir
Copy link
Collaborator

@tmyqlfpir tmyqlfpir commented Dec 29, 2024

Smaller enemies such as the hands, rats and fish appear to be designed in mind for autoaim as it's incredibly difficult to hit them precisely when autoaim is turned off.

This PR adds a new autoaim setting to allow players to always have autoaim set for the pitchfork weapon, so players who prefer not to use autoaim can have it only applied for the pitchfork.

test

@Hendricks266
Copy link
Member

My intuition would be, if I had autoaim disabled and I clearly missed but it hit anyway, that would not feel right. That said, I see that Blood enables autoaim unconditionally for the Voodoo Doll and the Life Leech.

In EDuke32, we make it so that if the crosshair is actually over the target when hitscan weapons are fired, they always hit the target regardless of random weapon spread, even if autoaim is disabled. This is because vanilla Duke 3D unconditionally uses autoaim, and the game was balanced in that way. In contrast, vanilla Blood has a setting to disable autoaim. This may be the true solution to what this PR intends to fix, and possibly even the forced autoaim in the original game.

@tmyqlfpir
Copy link
Collaborator Author

I believe the reason why Voodoo Doll and Life Leech both force autoaim is that the weapon is designed around that.

The pitchfork isn't at all, but I feel it should be given that it is used at a point blank range and should be forgiving rather than relying precise aiming, unlike using an hitscan weapon at a greater distance. It is also disabled for vanilla mode for the sake of accuracy.

@ElJulo25
Copy link

I prefer no auto aim imho

@tmyqlfpir tmyqlfpir marked this pull request as draft January 1, 2025 02:16
@tmyqlfpir tmyqlfpir marked this pull request as ready for review January 1, 2025 02:23
@tmyqlfpir tmyqlfpir changed the title Blood: Always autoaim for pitchfork Blood: Add autoaim option for pitchfork Jan 1, 2025
@tmyqlfpir tmyqlfpir changed the title Blood: Add autoaim option for pitchfork Blood: Add 'pitchfork only' autoaim option Jan 1, 2025
@Hendricks266 Hendricks266 merged commit a8c7eed into NBlood:master Jan 7, 2025
10 checks passed
@tmyqlfpir tmyqlfpir deleted the pitchfork-autoaim branch January 7, 2025 05:16
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.

3 participants