FastAsyncWorldEdit

FastAsyncWorldEdit

152k Downloads

Bad calculation of the number of blocks changed?

jewome62 opened this issue ยท 3 comments

commented

Server Implementation

Paper

Server Version

1.18.2

Describe the bug

I have currently some problems with max-changed limitation.

I have a rank named 'builder-plus' with 8 millions max-change

limits:
[some-line]
  builder-plus:
    # Max actions that can be run concurrently (i.e. commands)
    max-actions: 2
    # Max number of block changes (e.g. by `//set stone`).
    max-changes: 8000000
    # Max number of blocks checked (e.g. `//count stone` which doesn't change blocks)
    max-checks: 80000000
    # Number of times a change can fail (e.g. if the player can't access that region)
    max-fails: 4000000
    # Allowed brush iterations (e.g. `//brush smooth`)
    max-iterations: 500
[some-line]

But with //pos1 //pos2 and //set stone they are limited to 2.5 millions blocks.

I have checked, the permission is well configured, here exemple of luck perms record:
image

To Reproduce

  1. Define limitation
  2. Check permission to be limited
  3. Try the limitation with simple //set stone

Expected behaviour

I suppose there are complicated to fix this kind of bug.

Many add some step into resolution of this bug :
As add log or complete the error message with value compared (current max-changed of player and current selection count) ?

Screenshots / Videos

image

Error log (if applicable)

No response

Fawe Debugpaste

https://athion.net/ISPaster/paste/view/6e996039991c46ef9176dc2941395c5c

Fawe Version

FastAsyncWorldEdit-2.4.1

Checklist

Anything else?

No response

commented

Hi,

The same in 1.19.1 and the last version.

commented

Same here, the plugin doesn't seem to use the max-changes value correctly
FAWE debugpaste: https://athion.net/ISPaster/paste/view/35e51bf1a01d4b5cb86f2b692362a452
In this test I have max-changes: 2000000 and max-checks: 2000000 but I'm trying to do a worldedit operation that will only change 1.5 million blocks and it doesn't let me:
2022-08-16_14 25 42

A temporary fix is to just set the number way higher

commented

I don't think the max-changes feature has ever worked correctly to be honest.