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

TempEntities in S2 #72

Open
fischerq opened this issue Jan 25, 2016 · 2 comments
Open

TempEntities in S2 #72

fischerq opened this issue Jan 25, 2016 · 2 comments

Comments

@fischerq
Copy link
Contributor

I believe the TempEntities got completely removed in S2 replays (or to be precised, moved into DataUserMessages).
It might be good to remove the TempEntity processor, or at least indicate that it is deprecated.

References:

  • I ran into the issue when I tried to use the OnTempEntity event and it did not fire at all.
  • The net messages for TempEntities got removed from S1 to S2
  • New entries in the DotaUserMessages
@spheenik
Copy link
Member

You're right, I figured instead of deprecating, I'd better supply the new ones with a common interface (like combat log). Just have not come around to doing it yet.

Is there any useful information to get from temp entities?

@fischerq
Copy link
Contributor Author

Temp entities are primarily used for projectiles and animations. I'm planning to use it for quite a lot, i.e. timing (and targeting) of attacks, animation canceling, dodging of hits/spells.

I haven't looked into the specific fields available in the S2 ones, but I guess it didn't change much.

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

No branches or pull requests

2 participants