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

Eating won't work (heroku) #442

Open
PortPixel opened this issue Jul 8, 2016 · 14 comments
Open

Eating won't work (heroku) #442

PortPixel opened this issue Jul 8, 2016 · 14 comments

Comments

@PortPixel
Copy link

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)

@ghost
Copy link

ghost commented Jul 8, 2016 via email

@PortPixel
Copy link
Author

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
Copy link
Author

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

@ghost
Copy link

ghost commented Jul 9, 2016 via email

@PortPixel
Copy link
Author

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

@valonymeri
Copy link

Is this fixed yet ?

@noahcoetsee
Copy link

Yes.

@noahcoetsee
Copy link

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.

@matthall1998
Copy link

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

@baileypumfleet
Copy link

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.

@Ayelis
Copy link

Ayelis commented Nov 13, 2017

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

@noahcoetsee
Copy link

noahcoetsee commented Nov 13, 2017

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

@Ayelis
Copy link

Ayelis commented Nov 13, 2017

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

@noahcoetsee
Copy link

Supposedly, it is fixed :D

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

6 participants