Realistic Terrain Generation

Realistic Terrain Generation

3M Downloads

Half-cobble/half-sandstone villages

chaNcharge opened this issue ยท 26 comments

commented

Minecraft 1.7.10
RTG 0.8.0 RC-2
Seed: 3029737
Loaded on BiomeTweaker with BOP RTG tweaks
Coordinates on top right
Only happens sometimes, hard to reproduce
2016-05-14_13 25 21

commented

The world looks completely different to me, with the same versions, seed and location.

  1. Is the seed correct?
  2. Did you tweak the BiomeTweaker's RTG preset configs? Or any configs?
  3. Which BiomeTweaker's version do you use? I tested with the latest Release, "BiomeTweaker-1.7.10-1.2.128"
    Edit: Tested with the latest Beta too, the area is still Savanna and not Desert there.
commented

@chaNcharge Can you supply a fml-client-latest.log file so we can investigate further?

commented

Yeah, I did get the seed and the villages to work (they didn't generate with your settings, I had to lower the min/max values), but I can't reproduce, villages work fine for me near biome borders, where the issue is.
2016-05-18_14 21 46

It may indeed happen but is hard to reproduce, like you said. Without that I can't do much though I'm afraid. The issue has been resolved for the vast majority of the cases, (I flew around quite a bit and didn't see anything) I'm not sure that spending more time on this would be beneficial. I'll still leave this open in case some other dev manages to reproduce it.

Thank you for reporting it though @chaNcharge, I've seen you quite a lot around here, your help is appreciated!

commented

The seed definitely is correct, I did change the RTG mod config, village sizes have been increased to 1 and village spacing are the default values. I did have BOP installed on 1.7.10. Every time I regenerated this world, it was always a desert, no matter what. Make sure you're teleporting to the right coordinates. I'll see if I can get a log file, I'll post it as soon as I have it.

commented

I'm using the dev version, but yes it is hard to reproduce. I'm on a phone right now so I can't send a log yet, but I'll try to get one soon.

commented

So I regenerated the world for what seems like 15 times, and I managed to sort of reproduce the bug. It's not quite the bug I was describing, but definitely a step in the right direction.
2016-05-18_22 46 46
Does it seem sort of off to the overall theme of the village? It is not a separate village, it's inside just one village. Here's the fml-client-latest.log file: LOOK AT EDIT FOR NEW FILE

EDIT: We got one! (Sorry for ghostbusters reference) I reloaded the world a few more times, and this looks much more serious now. It took 17 regenerations to finally reproduce this bug. Here's the new log file: https://drive.google.com/file/d/0B3ZoX-8LP704LWVwRU9za1BrdGs/view?usp=sharing
2016-05-18_23 17 52
2016-05-18_23 18 12

commented

I dont htink theres a lot to do about this - we are fairly limited by those forge events

commented

If we're using MapGenVillageRTG, can we set it on a village placement check?

commented

yes, but we'd need to overrwrite StructureVillagePieces, which might be a bad idea

commented

I was always quite reluctant to do our own StructureVillagePieces, but it might be worth a shot if it's going to totally resolve this issue.

My only concern is that it's going to cause compatibility issues with other village-related mods somehow, though I don't know if it would.

In any case, if we did try to write our own StructureVillagePieces, what would we need to do in terms of a 'placement check'? Do you mean checking the biome from within StructureVillagePieces before deciding which block to place, or something different?

commented

just checking it when the village is started, and then passing it down to the rest of the things

commented

Isn't that what we're already doing with the Forge event? We're checking the biome during preDecoration and then passing it to the Forge event, but... that doesn't seem to work 100% of the time.

How will passing it to StructureVillagePieces be any different?

commented

because the biome forge is passing is weird... i think its on a per house scale

commented

It's worth a shot I guess.

Let's look at it after 0.8 is released ๐Ÿ‘

commented

I think this is rare enough that we can leave it until after 1.0 is released.

commented

Seems to be more common in RTG 1.0 snapshot 2, got the bug to reproduce TWICE in the same world after 1 world regeneration compared to originally 10-15. Same seed, same location, although I increased village size to 1 (from 0) and village spread to vanilla values.
2016-06-19_22 33 38
2016-06-19_22 47 35

commented

Just tested this in the official 1.0 release, still present

commented

@chaNcharge, can you pastebin/gist the list of mods you are using?
Also, what is the frequency that you observe this happening?

I did some testing a couple of days ago with a small group of mods that alter villages and out of ~100 villages observed I only found 1 that had a single broken building, similar to as you reported here: #686, but I didn't observe anything like in the screenshots in this issue.

commented

@srs-bsns is that broken building reproducible? Mine (also rare) have not been.

commented

It doesn't happen too often, but this also occurs by itself and biomes o plenty with the RTG BOP tweaks

commented

@Zeno410, I didn't attempt to reproduce it, and sadly I no longer have the config setup to go back and try.
However, for future reference on 'cut off buildings' (probably not related to this issue): MinecraftForge/MinecraftForge#3020

commented

If this is the same Issue as the race condition in Forge (#3020) then the fix for it (MinecraftForge/MinecraftForge#3061) may fix this issue, but unfortunately the fix will only be in the latest Forge. Maybe there's a way for us to incorporate the fix in RTG to prevent this?

commented

My take is no, we can't do it. We'd have to hack around in critical MC/Forge operations (the Anvil loader itself, ick)

commented

Oh well, If we can't, it's not a big deal. I am mostly concerned with moving forward, so hopefully this issue is caused by the same double chunk gen race condition that is causing #686. We atleast shouldn't see this sort of thing in RTG 2.. and beyond.

commented

Should be fixed in 625b591 - closing until it rears its ugly head again.

commented

@chaNcharge
I have not observed this happening once, in hundreds of test worlds created, so I still feel that this is either a mod interaction bug, or possibly MinecraftForge/MinecraftForge#3020, or a combination of both. In any case there isn't anything we can really do about it.

In the case that this really was RTG's fault, then I know exactly where the problem was. It was in code added to fix #392, which has just been removed because it no longer seems to be relevant. But let us know if you still observe this happening.