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

fix laning #191

Closed
4 of 5 tasks
Keithenneu opened this issue Mar 28, 2017 · 6 comments
Closed
4 of 5 tasks

fix laning #191

Keithenneu opened this issue Mar 28, 2017 · 6 comments

Comments

@Keithenneu
Copy link
Contributor

Keithenneu commented Mar 28, 2017

Our safe lane is dying a lot. Most of it comes from over aggression / bad positioning.
They also miss a few easy last hits.

Not saying that it's all bad, but I think it really needs an update.

This is mostly from observing a CM/PA safe lane. should be true for other lanes and combinations.
Loosely ordered by severity.

  • don't do a full retreat if staying back a little is all we need to do (let regen items do their work)
  • manage creep aggro while harassing
  • don't walk into the enemy tower, unless we control aggro
  • cm frost bites random creeps. not sure if she's trying to last hit (not really worth on lane creeps) or trying to harass (missclick??)
  • improve creep health prediction (would probably require a simulation at this point)

I'm also planning some risk assessment for lanes. Might help with the positioning later on.

@Nostrademous
Copy link
Owner

I implemented some fixes to address some of these. working on smarter in-lane harassment currently.

@Nostrademous
Copy link
Owner

okay, I just pushed changes that do a lot of the above.

Currently I forced hero selection to only 5 heroes that have proper nukeDamage calculations implemented. Works great. Give it a go.

@Keithenneu
Copy link
Contributor Author

Aw yeah, much better!

Just more edge cases I noticed (I'm kinda abusing the issue tracker as a public note pad ath this point..):

  • if we are pushing and our creeps are getting pulled behind the tower, bots just stand in the tower and tank it without attacking back or anything.
  • cm should use her frostbite if she's on full mana and is somehow able to reach an enemy. That'd allow drow to get some right clicks in. In general: as long as we have enough mana to handle an emergency, we can as well use it to hurt someone. (BS will appreciate low enemies :) )
  • If an enemy dives the tower, slow him, frostbite him, kill him!

@Nostrademous
Copy link
Owner

Nostrademous commented Mar 29, 2017

CM does use frostbite if above a certain % mana threshold on nearby "weakest" enemy (providing the non-weakest enemy is not standing in the way, and the distance from her current position is less than Frostbite CastRange + small delta).

@Nostrademous
Copy link
Owner

  • If an enemy dives the tower, slow him, frostbite him, kill him!

That code used to exist in the utils.HarassEnemy() ... but I commented it out or scrapped it b/c it didn't belong there as it wasn't a harassment. I need to re-create it as part of the "fight" mode.

@Nostrademous
Copy link
Owner

  • if we are pushing and our creeps are getting pulled behind the tower, bots just stand in the tower and tank it without attacking back or anything.

Yeah, I know... it was hard enough getting them to not go past the tower... bots and understanding positioning is a hard problem. We need a generic solution that will tie into your map-wide orientation.

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

No branches or pull requests

2 participants