Skip to content

[Highly Customized] New AdvancedDrive locomotor #1622

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

[Highly Customized] New AdvancedDrive locomotor #1622

wants to merge 18 commits into from
+2,169 −19

Conversation

CrimRecya
Copy link
Contributor

@CrimRecya CrimRecya commented Apr 10, 2025 •
edited
Loading

  • AdvancedDrive locomotor ({4A582751-9839-11d1-B709-00A024DDAFD1}) is an improved Drive locomotor ({4A582741-9839-11d1-B709-00A024DDAFD1}) that can serve as a complete upper level replacement for it.
  • It fixes the issue where Drive cannot correctly crush objects during rapid turns.
  • It has smoother uphill and downhill dynamic visual effects.
  • It has the function of driving the unit backwards.
    • AdvancedDrive.ReverseSpeed controls the speed ratio when reversing.
    • AdvancedDrive.FaceTargetRange controls how close the unit is to its target, allowing reversing.
    • AdvancedDrive.ConfrontEnemies controls whether to maitain the frontal movement towards the enemy within the aforementioned distance and no longer automatically selects by the current orientation.
    • AdvancedDrive.RetreatDuration controls how long since the unit was last injured, allowing reversing.

In rulesmd.ini:

[SOMEVEHICLE]                  ; VehicleType
Locomotor=AdvancedDrive              ; Locomotor
AdvancedDrive.ReverseSpeed=0.85      ; floating point value
AdvancedDrive.FaceTargetRange=16.0   ; floating point value
AdvancedDrive.ConfrontEnemies=true   ; boolean
AdvancedDrive.RetreatDuration=150    ; integer, game frames

@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 10, 2025 •
edited
Loading

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@Coronia Coronia added Needs testing ⚙️T2 T2 maintainer review is sufficient labels Apr 10, 2025
@CrimRecya
Copy link
Contributor Author

CrimRecya commented Apr 11, 2025 •
edited
Loading

The reverse function, when used in conjunction with #1623 and #1480, can provide a more complete gaming function. While when used in conjunction with KeepTargetOnMove and KeepRange, it can enhance the player's gaming experience.
reverse

@Starkku
Copy link
Contributor

Starkku commented Apr 11, 2025

Irregardless of any other factors (I have not had time to do in-depth review yet), the name probably needs to be something less ambiguous than Skilled.

@CrimRecya
Copy link
Contributor Author

Irregardless of any other factors (I have not had time to do in-depth review yet), the name probably needs to be something less ambiguous than Skilled.

Well, it's indeed not quite clear. Do you have any suggestions?

@Metadorius
Copy link
Member

AdvancedDrive?

Metadorius
Metadorius reviewed Apr 11, 2025
View reviewed changes
Copy link
Member

@Metadorius Metadorius left a comment

Choose a reason for hiding this comment

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

Marked all the places I found where something could be named or brought to YRpp for your convenience.

src/Locomotion/SkilledLocomotionClass.h Outdated
{
const auto pLinked = this->LinkedTo;

if (!pLinked->unknown_abstract_array_588.Count)
Copy link
Member

Choose a reason for hiding this comment

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

and here

src/Locomotion/SkilledLocomotionClass.cpp Outdated
if (pLinked->OnBridge != pNextCell->ContainsBridge())
pLinked->unknown_bool_68B = true;

if (!pLinked->vt_entry_29C())
Copy link
Member

Choose a reason for hiding this comment

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

and here

@CrimRecya
Copy link
Contributor Author

AdvancedDrive?

At first, I thought of SkilledDrive, but I felt it was too long, so I changed it to the current one. But obviously, this is confusing.

@CrimRecya CrimRecya changed the title [Highly Customized] New Skilled locomotor [Highly Customized] New AdvancedDrive locomotor Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Metadorius Metadorius Metadorius left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Needs testing ❓New feature ⚙️T2 T2 maintainer review is sufficient
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@CrimRecya @Starkku @Metadorius @Coronia @TaranDahl