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.

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

LoadCargoBotModule to replace most of AI preload units #53

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

LoadCargoBotModule to replace most of AI preload units #53

wants to merge 3 commits into from

Conversation

dnqbob
Copy link
Contributor

@dnqbob dnqbob commented May 7, 2023 •
edited
Loading

This module make AI be able to load their units into transport

@dnqbob dnqbob changed the title LoadCargoBotModule to replace most of AI preload unit LoadCargoBotModule to replace most of AI preload units May 7, 2023
@dnqbob
Copy link
Contributor Author

dnqbob commented May 7, 2023 •
edited
Loading

The benefit of this module:

  1. Yaml code simply. Most of preloaded transport for AI is no longer needed, makes the code more readable and maintainable. (I don't remove them in this PR, in case of crashing the campaigns)
  2. Al will load the passengers once again instead of leaving them to the ground.
  3. Ease the bug (when preloaded passenger changes weapon on upgrade, AttackGarrison is not able to sync with the change) on AttackGarrison of preloaded.

@dnqbob dnqbob force-pushed the cargo branch 2 times, most recently from d2a62d1 to 349c7de Compare August 24, 2023 03:50
@dnqbob
Copy link
Contributor Author

dnqbob commented Aug 24, 2023

Add a setting to allow AI load unit for each other, so there will be a chance of mixed tech units with more funs.
mixed-tech-cargo

@dnqbob dnqbob force-pushed the cargo branch 2 times, most recently from 1e611e6 to fb3cbde Compare February 5, 2024 02:32
@dnqbob dnqbob force-pushed the cargo branch 4 times, most recently from fd6fdca to dbef600 Compare February 16, 2024 09:48
@dnqbob dnqbob force-pushed the cargo branch 2 times, most recently from fd3441a to 88f9b8e Compare February 26, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dnqbob