agar.io-clone icon indicating copy to clipboard operation
agar.io-clone copied to clipboard

Eating won't work (heroku)

Open PortPixel opened this issue 8 years ago • 14 comments

Eating won't work properly - it seems you have to wait under a cell for ~10 seconds to be eaten (this may just be certain positions though)

PortPixel avatar Jul 08 '16 09:07 PortPixel

I think that you can only eat a cell when the middle of both cells are on exactly the same position…

ghost avatar Jul 08 '16 11:07 ghost

That what I though (a commit I saw at same point updated the code to use the default player mass as the radius of cells, before this eating didn't work at all). I'll look though the code when I get a computer (I'm on my phone now) and try to fix it

PortPixel avatar Jul 09 '16 19:07 PortPixel

I don't think it has to be the exact same position, just a very small radius/only a certain distance perhaps?

PortPixel avatar Jul 09 '16 19:07 PortPixel

Yes, that was my commit. It was a really quick fix for making eating work again. I should really update it to change the radius to the current player mass.

ghost avatar Jul 09 '16 19:07 ghost

Is anyone planning to fix this? I have tried but it didn't seem to work

PortPixel avatar Jul 12 '16 17:07 PortPixel

Is this fixed yet ?

valonymeri avatar Mar 28 '17 10:03 valonymeri

Yes.

noahcoetsee avatar Mar 30 '17 00:03 noahcoetsee

The reason that some things are not eaten properly, is mainly because the radius calculations are off, and you have to be larger than one would originally think to consume another cell. This can be easily fixed by changing the calculations, however, in essence, yes it has been fixed.

noahcoetsee avatar Mar 30 '17 00:03 noahcoetsee

How does one change these calculations? Be cool to put it in the config file to make it an easy change.

matthall1998 avatar May 31 '17 17:05 matthall1998

I remember having this problem a little while ago. I believe that you need to go into the src/server/server.js file and you need to find the collision lines and manually increase the radius of the collision boundary.

baileypumfleet avatar Jun 07 '17 06:06 baileypumfleet

Is this also a duplicate of #413? And is #432 related somehow?

Ayelis avatar Nov 13 '17 08:11 Ayelis

Is this also a duplicate of #413? And is #432 related somehow?

Not exactly, #413 is more about an actual error (if you scroll down a bit), and #423 is a fix for that error. This is more about the default configurations being a little messed up, and it is really based on preference. I think this was originally a problem, then it was fixed and just became a configurable attribute in server.js

noahcoetsee avatar Nov 13 '17 15:11 noahcoetsee

Ah, but if 413 was truly fixed by 432, wouldn't they both be closed by now? :o ;)

Ayelis avatar Nov 13 '17 17:11 Ayelis

Supposedly, it is fixed :D

noahcoetsee avatar Nov 16 '17 13:11 noahcoetsee