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

ancient exp why not prioritize use ancient trees? #476

Open
jzy-chitong56 opened this issue Dec 19, 2024 · 3 comments
Open

ancient exp why not prioritize use ancient trees? #476

jzy-chitong56 opened this issue Dec 19, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@jzy-chitong56
Copy link
Contributor

If WISP is always used as a priority every time, then this feature loses its uniqueness and many idle ancient trees are left unused

In my branch, I will prioritize the use of ancient trees, and they are the idle ancient trees closest to the expansion location

@SMUnlimited
Copy link
Owner

Unfortunately no easy way to determine if ancient is rooted or not, so you are very likely you end up uprooting existing expansions instead.

Also may take ages to move a tree from base to expansion location so its faster to just send a wisp.

@SMUnlimited SMUnlimited added the enhancement New feature or request label Dec 20, 2024
@jzy-chitong56
Copy link
Contributor Author

https://github.com/jzy-chitong56/AMAI-CN/blob/master/Jobs/ANCIENT_EXPANSION.eai

As you said, it is indeed very difficult, but this is the most distinctive feature of the entire function. Even if it is placed in the special expansion of the race, it will not affect normal expansion. My idea is that the tree does not need protection, just leave it to crawl quietly by itself. Of course, creep need to be cleared out in advance

@jzy-chitong56
Copy link
Contributor Author

call CaptainAttack(GetUnitX(ancient_exp_unit), GetUnitY(ancient_exp_unit))

Actually, it doesn't make sense to move to the side of the tree, but the following command will still keep them away from the tree

call AttackMoveKillCreep(ancient_exp_creep, false)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants