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

Tweaks to the categories of various blueprints #6006

Merged
merged 10 commits into from
Mar 23, 2024

Conversation

Basilisk3
Copy link
Collaborator

@Basilisk3 Basilisk3 commented Mar 19, 2024

Description of the proposed changes

  • Add the ANTIMISSILE category to the Iavish (Seraphim Tech 3 Aircraft Carrier).
  • Removed the MASSFABRICATION category from all ACUs, SACUs and the Paragon. They retain the MASSPRODUCTION category.

Is there any reason why SACUs might need the MASSFABRICATION category?

Edit: Apparently this category is not vital for these units.

Checklist

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

@Garanas Garanas added the area: unit categories related to unit categories defined in the blueprint files label Mar 20, 2024
@Garanas
Copy link
Member

Garanas commented Mar 20, 2024

Perhaps they originally were mobile mass fabricators. There have been numerous 'redesign' left overs in the code base 😃

@Basilisk3 please add a changelog entry in 3804.md

@Garanas
Copy link
Member

Garanas commented Mar 21, 2024

  • Removed the SNIPER category from the Megalith.

This is an interesting one - it is actively used by AIs to identify the Megalith as an experimental that can act like a sniper. If I'm not mistaken then @maudlin27 does this

@Basilisk3
Copy link
Collaborator Author

  • Removed the SNIPER category from the Megalith.

This is an interesting one - it is actively used by AIs to identify the Megalith as an experimental that can act like a sniper. If I'm not mistaken then @maudlin27 does this

I see. If this causes trouble for the AI, then I will revert the change.

@lL1l1
Copy link
Contributor

lL1l1 commented Mar 21, 2024

The SNIPER category also makes t2 artillery prefer to target Megalith over lower range experimentals.

TargetPriorities = {
"EXPERIMENTAL MOBILE UEF ARTILLERY",
"(ARTILLERY - TECH1)",
"ANTISHIELD",
"SNIPER",
"SHIELD STRUCTURE",
"EXPERIMENTAL MOBILE",
"TECH3 ENERGYPRODUCTION",
"TECH2 ENERGYPRODUCTION",
"RESEARCH",
"TECH3 MASSPRODUCTION",
"TECH2 MASSPRODUCTION",
"TECH3 MOBILE",
"TECH2 MOBILE",
"(STRUCTURE * DEFENSE - ANTIMISSILE)",
"COMMAND",
"TECH1 MOBILE",
"ALLUNITS",
},

@Garanas Garanas merged commit ea2950e into FAForever:deploy/fafdevelop Mar 23, 2024
@Basilisk3 Basilisk3 deleted the Categories2 branch March 29, 2024 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: unit categories related to unit categories defined in the blueprint files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants