/function build:every_nether is breaking the Arena rebuilding

[AllYourBlocks]

This is an expansion on issue #178 which I will close as there is information provided here...

When /function build:every_nether has been used, the Arena does not rebuild correctly afterwards when using /function rebuild:arena

Behaviour is... Yellow Side builds... Blue side does not build, apart from Blue Home Chunk which does build... then wool does get painted correctly

Running /function build:blue_nether and /function build:yellow_nether separately does not cause the subsequent /function rebuild:arena command to fail on Blue side

[AllYourBlocks]

Notably, in Random Condensed, executing /function build:every_nether did not break the Arena rebuild afterwards

The number of commands run was different... 480 vs 490 (the standard number) on occasion, unclear why this is

[AllYourBlocks]

I also note that the number of commands run in /function build:every_nether is substantially more than the sum total number of commands in /function build:blue_nether and /function build:yellow_nether

[slipcor]

I think Mojang did a few changes to chunk loading or unloading order, and thus now it became a problem that we do all the painting of chunks in one tick, and then also in the same tick move the markers away.

Especially when we do multiple paintings (which happens at "every nether") this becomes a bug.

I could recreate the error simply by running the blue nether function - which made no sense to me, somewhere along the way I might have just fixed it now.

Also, the red and green nether-functions were always completely called even with 2 team mode. This can definitely have messed up the timings, so much so that we had 4 threads fighting over the paste markers going back and forth between pasting another nether chunk and getting reset before the next color tries to paste its nether chunks, somehow losing the paster markers in the process.

Fact of the matter is, with any function apart from "build yellow", we ended up deleting 2 out of the 4 markers. Now all 4 markers persist. Let us examine this further. maybe I need to rewrite the marker safety feature.

[slipcor]

Can be tested now, should be fixed in latest commit

[slipcor]

To explain, this issue was caused by two things at the same time:

First issue: Attempt to build green and red nether

We were calling red and green nether build functions regardless of how many teams we actually need. I assumed this can do no harm. I was wrong.

This is the code we used to have, it will always be called, when asking to build either every nether or in this case green:

ChunkWars/chunkwars/data/arena/functions/build/green_nether.mcfunction

Line 4 in f5e502d

execute if score $map_condensed map_variation matches 0 run function arena:build/4teams/modern/standard/green_nether

Now let us assume the highlighted line is the one we call, we are in a 2 team arena, modern, not condensed. This will call this function:

ChunkWars/chunkwars/data/arena/functions/build/4teams/modern/standard/green_nether.mcfunction

Line 9 in f5e502d

function arena:build/4teams/modern/condensed/raw/270_4

Here the highlighted function attempts to build the green nether in location of the 4 teams modern standard arena, thus teleporting the green marker out of bounds, unloading it. Furthermore, the function function arena:paint/4teams/modern/standard - at the end - moves ALL markers to the mid point of the 4 teams modern standard arena, effectively trapping all markers. It is random and lucky that any markers survive this and come back by the painting that we do in the proper required yellow/blue call.

First fix: Verify that we are in a 4 team arena before even trying to deal with green and red nether:

ChunkWars/chunkwars/data/arena/functions/build/green_nether.mcfunction

Line 4 in 8538c2d

execute if score $teams gamestate matches 4 if score $map_condensed map_variation matches 0 run function arena:build/4teams/modern/standard/green_nether

Check that we are in 4 teams mode, otherwise skip green and red nether building.

Second issue: 4 team standard arena actually attempted to build condensed arena chunks

Here is the example with red, but it was copypasted wrongly for all colors:

ChunkWars/chunkwars/data/arena/functions/build/4teams/modern/standard/red_nether.mcfunction

Line 9 in 8538c2d

function arena:build/4teams/modern/condensed/raw/0_4

So we moved the build-markers over into the wrong arena, unloading them.

Red is an even more special case because you might see it is calling "0_4" which is the chunk with index 4 - correct - but with orientation "no rotation offset", which is wrong because that is the blue chunk. So the red nether tried to build the blue chunk. This is now also fixed.

Second fix: call proper functions of standard arena, and use actual red chunk for red nether