ChunkSpawnerLimiter

17 - Limiter doesn't work on different heights in a chunk

While writing this ticket I realised this is not a bug, so I've made it an Enhancement.

What steps will reproduce the problem?
1. Make like 8 mobspawners spawning pigzombies on y=146 in 1 chunk (exact situation from what caused lag on our server).
2. Let them drop to a height of 126.
3. Wait and watch the TPS drop and the mobs increase on the chunk ('/lagg chunk' from clearlagg was a good tool for me).

What is the expected output? What do you see instead?
Expected: the plugin will watch the chunk on full height.
Instead: It seems that the plugin uses 16*16*16 chunks, while I thought otherwise.

What version of the product are you using?
CSL 2.10.0

Do you have an error log of what happened?
Nope.

Please provide any additional information below.
The limiter does work when the chunks are on the same height.
Maybe you could add an option to watch the full chunk (16*16*256), or only a part of it (16*16*16, which is probably used right now).

Here is my config:


  1. ANIMAL:
  2. totalPerChunk: 30
  3. MONSTER:
  4. totalPerChunk: 20
  5. WATER_MOB:
  6. totalPerChunk: 5
  7. AMBIENT:
  8. totalPerChunk: 5
    checkSurroundingChunks: true
    surroundingRadius: 2
    onlyLimitSpawners: true
    excludedWorlds:
    - NoLimitsWorld
    mobs:
    ANIMAL:
    totalPerChunk: 250
    MONSTER:
    totalPerChunk: 250
    WATER_MOB:
    totalPerChunk: 80
    AMBIENT:
    totalPerChunk: 100
    debuggingMode: false
    checkNewVersionOnStartup: false

I think increasing the amount of chunks to check around 1 chunk will take more resources than when you check a chunk on full height.

User When Change
Cyprias Mar 03, 2014 at 09:52 UTC Changed status from New to Fixed
TheFarmerNL May 21, 2013 at 11:00 UTC Changed description:
  Please provide any additional information below.
  The limiter does work when the chunks are on the same height.
- [b]Maybe you could add a config option to watch the full chunk (16*16*256), or only a part of it (16*16*16, which is probably used right now).[/b]
+ [b]Maybe you could add an option to watch the full chunk (16*16*256), or only a part of it (16*16*16, which is probably used right now).[/b]
  Here is my config:
  #
TheFarmerNL May 21, 2013 at 10:45 UTC Create

You must login to post a comment. Don't have an account? Register to get one!

Facts

Last updated
Mar 03, 2014
Reported
May 21, 2013
Status
Fixed - Developer made requested changes. QA should verify.
Type
Enhancement - A change which is intended to better the project in some way
Priority
Medium - Normal priority.
Votes
0

Reported by

Possible assignees