ensingm2 / SteamMonsterGameScript

A Javascript automator for the 2015 Summer Steam Monster Minigame
78 stars 29 forks source link

Disable ability use particles? #114

Closed cdmichaelb closed 9 years ago

cdmichaelb commented 9 years ago

They aren't creating memory leaks as far as I can tell, but they are having a significant impact on performance (especially with multiple slaves open). Can those particles also be disabled?

ensingm2 commented 9 years ago

They aren't creating memory leaks as far as I can tell, but they are having a significant impact on performance

I'll look into an option to disable ability particles

(especially with multiple slaves open)

I don't think this should be the case. See This renderer disabling code, As the renderer is deleted, I don't think any particles should spawn in slaves. Are you sure it's not just the extra computation from the extra clicking/scripts?

cdmichaelb commented 9 years ago

It could be the script, I meant manual slaves though (since as discussed in other post I'm fairly certain the generated slaves don't work, and i'm 50/50 on manual slaves working), probably moot since slaves probably don't work at all.

But the particles are still an issue, even for a single client especially on early levels, so it'll be nice to have on new days.

vanZeben commented 9 years ago

Quickfix for effect particles, we can simply "CSceneGame.prototype.SpawnEmitter = function(){}", mind you that will start throwing errors since things aren't being caught correctly and an ability has to be inactive in the lane after you type this (as in it has to go inactive) before it will stop showing it, but if you dont care about console errors, throw that in your console

vanZeben commented 9 years ago

SteamDB's implementation just straight up disables the whole renderer and simulates 1fps themselves... We could merge that in?

vanZeben commented 9 years ago

See #148