search

Civcraft / MusterCull

A Bukkit plugin and Minecraft mob limiter.
BSD 3-Clause "New" or "Revised" License
2 stars 6 forks source link

Mobs aren't spawning. This might help determine why. #15

Closed CodingMonkeyXYZ closed 9 years ago

CodingMonkeyXYZ commented 9 years ago

https://bugs.mojang.com/browse/MC-42053

As I mentioned to ttk, I ran a test on a vanilla MC server in 1.8 with a low chunk loading distance and my mob grinder suddenly just stopped producing mobs. I increased the chunk loading distance again and it magically fixed it.

This issue is new to 1.7 I think.

I'm not necessarily suggesting we permanently change the mob cap but if a five minute test helps us determine where, in the code, the lack of spawns in coming from, might it be worth testing this? This could at least be part of the problem.

ttk2 commented 9 years ago

whats the suggestion bump the view distance to 10?

On Mon Feb 16 2015 at 4:23:00 AM Simian Axemeister notifications@github.com wrote:

https://bugs.mojang.com/browse/MC-42053

As I mentioned to ttk, I ran a test on a vanilla MC server in 1.8 with a low chunk loading distance and my mob grinder suddenly just stopped producing mobs. I increased the chunk loading distance again and it magically fixed it.

This issue is new to 1.7 I think.

I'm not necessarily suggesting we permanently change the mob cap but if a five minute test helps us determine where, in the code, the lack of spawns in coming from, might it be worth testing this? This could at least be part of the problem.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/MusterCull/issues/15.

CodingMonkeyXYZ commented 9 years ago

Yes, if only to run a simple test to confirm this causes the issue. If it does, that's something to go on at least.

EDIT: to be clear: do this and then go to a vanilla mob farm or a slime chunk and compare results before and after, for instance.

ttk2 commented 9 years ago

that's some what difficult to do if we only restart once that day, I will have to make a point to restart the server, what view distance to the problems supposedly start at?

On Mon Feb 16 2015 at 2:51:46 PM Simian Axemeister notifications@github.com wrote:

Yes, if only to run a simple test to confirm this causes the issue. If it does, that's something to go on at least.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/MusterCull/issues/15#issuecomment-74570623.

CodingMonkeyXYZ commented 9 years ago

I think the problem occurred when I had a view distance lower than 10. I have it set to 10 now on my server and everything works fine.

ttk2 commented 9 years ago

that's not a solution a view distance of 10 is a lot more than 10 times the server load, that's hundreds of chunks per player, view distance 4 is 64 chunks per player.

On Wed Feb 18 2015 at 8:33:52 AM Simian Axemeister notifications@github.com wrote:

I think the problem occurred when I had a view distance lower than 10. I have it set to 10 now on my server and everything works fine.

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/MusterCull/issues/15#issuecomment-74872762.

CodingMonkeyXYZ commented 9 years ago

I didn't mean you should leave it at that but if it show that this improves spawning then it would indicate where the problem might be occurring in the code. If you don't want to run this test on CivCraft, I totally understand but perhaps you (or someone else) could replicate the same experiment on CivTest, with a limited number of players?

ttk2 commented 9 years ago

I think civtest has a higher view distance already.

On Wed Feb 18 2015 at 8:38:52 AM Simian Axemeister notifications@github.com wrote:

I didn't mean you should leave it at that but if it show that this improves spawning then it would indicate where the problem might be occurring in the code. If you don't want to run this test on CivCraft, I totally understand but perhaps you (or someone else) could replicate the same experiment on CivTest, with a limited number of players?

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/MusterCull/issues/15#issuecomment-74873562.

TealNerd commented 9 years ago

view dist of 4 is 81 chunks actually

jjj5311 commented 9 years ago

this can be closed now?

ttk2 commented 9 years ago

yes

On Tue, Mar 17, 2015 at 9:12 AM jjj5311 notifications@github.com wrote:

this can be closed now?

— Reply to this email directly or view it on GitHub https://github.com/Civcraft/MusterCull/issues/15#issuecomment-82373769.