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

Adjust aurora handling #6486

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

lL1l1
Copy link
Contributor

@lL1l1 lL1l1 commented Oct 21, 2024

Description of the proposed changes

  • Boost aurora turn rate significantly so that the force it uses for movement can change direction instantly instead of forcing a turning maneuver.
  • Remove reverse speed so that it does not try to do half of the turn in reverse, as that gets it stuck when spamming move orders.

This makes it no longer get stuck for a short time doing a turning circle maneuver every time a move order backwards is given.
It does become harder to micro dodges since you can't dodge with the turning maneuver by occasionally clicking backwards/forward and instead have to click to the sides of the formation, which means that the aurora formation rotates sideways which puts your aurora out of range and in general messes with the overall formation. This more applies at larger counts though. At smaller amounts you don't have to deal with large formations so the desired sideways movement is easier to accomplish without messing up the formation badly, while also responding to commands as expected.

Testing done on the proposed changes

Use NoDamage in console, and spawn some auroras and a target to shoot at then move them around comparing the before and after of the changes, with various amounts of auroras and with spam clicking vs single clicking. Use ren_steering to show the steering (green means reverse).

Before:

aurora.movement.mp4

After:

aurora.movement.change.mp4

Additional context

Last time aurora movement was discussed was here: https://forums.faforever.com/viewtopic.php?f=52&t=3205

Checklist

- [ ] Changes are annotated, including comments where useful

  • Changes are documented in the changelog for the next game version

@lL1l1 lL1l1 added the area: balance idea related to suggestions for unit balance label Oct 21, 2024
@Garanas
Copy link
Member

Garanas commented Oct 21, 2024

That is an interesting alternative solution, very creative 😃 . I'll need to toy around with it to see the impact.

@lL1l1
Copy link
Contributor Author

lL1l1 commented Nov 18, 2024

Rather basic changelog snippet.

@lL1l1 lL1l1 marked this pull request as ready for review November 18, 2024 01:35
@lL1l1 lL1l1 added the area: balance related to units balance label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: balance idea related to suggestions for unit balance area: balance related to units balance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants