Skip to content

Commit

Permalink
Move more dungeon parameter selection to mapgens (#8653)
Browse files Browse the repository at this point in the history 
Move 'num_dungeons' to 'DungeonParams'.
Add new parameter 'num_rooms' to replace 'rooms_min' and 'rooms_max',
so that the mapgen has complete control over the number of rooms.
Add new bool 'first_room_large' so that the mapgen chooses this
instead of a hardcoded 1 in 4 chance.
Add new parameter 'room_size_large' to replace 'room_size_large_min'
and 'room_size_large_max', so that the mapgen has complete control
over this.
  • Loading branch information
@paramat
paramat committed Jul 9, 2019
1 parent 140aeab commit 0c31946
Show file tree
Hide file tree
Showing 4 changed files with 62 additions and 64 deletions.
35 changes: 14 additions & 21 deletions src/mapgen/dungeongen.cpp
View file
Expand Up @@ -64,10 +64,10 @@ DungeonGen::DungeonGen(const NodeDefManager *ndef,
dp.corridor_len_max = 13;
dp.room_size_min = v3s16(4, 4, 4);
dp.room_size_max = v3s16(8, 6, 8);
dp.room_size_large_min = v3s16(8, 8, 8);
dp.room_size_large_max = v3s16(16, 16, 16);
dp.rooms_min = 2;
dp.rooms_max = 16;
dp.room_size_large = v3s16(16, 16, 16);
dp.first_room_large = true;
dp.num_rooms = 16;
dp.num_dungeons = 1;
dp.notifytype = GENNOTIFY_DUNGEON;

dp.np_alt_wall =
Expand All @@ -76,10 +76,9 @@ DungeonGen::DungeonGen(const NodeDefManager *ndef,
}


void DungeonGen::generate(MMVManip *vm, u32 bseed, v3s16 nmin, v3s16 nmax,
u16 num_dungeons)
void DungeonGen::generate(MMVManip *vm, u32 bseed, v3s16 nmin, v3s16 nmax)
{
if (num_dungeons == 0)
if (dp.num_dungeons == 0)
return;

assert(vm);
Expand Down Expand Up @@ -119,7 +118,7 @@ void DungeonGen::generate(MMVManip *vm, u32 bseed, v3s16 nmin, v3s16 nmax,
}

// Add them
for (u32 i = 0; i < num_dungeons; i++)
for (u32 i = 0; i < dp.num_dungeons; i++)
makeDungeon(v3s16(1, 1, 1) * MAP_BLOCKSIZE);

// Optionally convert some structure to alternative structure
Expand Down Expand Up @@ -150,19 +149,14 @@ void DungeonGen::makeDungeon(v3s16 start_padding)

/*
Find place for first room.
There is a 1 in 4 chance of the first room being 'large',
all other rooms are not 'large'.
*/
bool fits = false;
for (u32 i = 0; i < 100 && !fits; i++) {
bool is_large_room = ((random.next() & 3) == 1);
if (is_large_room) {
roomsize.Z = random.range(
dp.room_size_large_min.Z, dp.room_size_large_max.Z);
roomsize.Y = random.range(
dp.room_size_large_min.Y, dp.room_size_large_max.Y);
roomsize.X = random.range(
dp.room_size_large_min.X, dp.room_size_large_max.X);
// Only the first room can be 'large'
if (dp.first_room_large) {
roomsize.Z = dp.room_size_large.Z;
roomsize.Y = dp.room_size_large.Y;
roomsize.X = dp.room_size_large.X;
} else {
roomsize.Z = random.range(dp.room_size_min.Z, dp.room_size_max.Z);
roomsize.Y = random.range(dp.room_size_min.Y, dp.room_size_max.Y);
Expand Down Expand Up @@ -204,8 +198,7 @@ void DungeonGen::makeDungeon(v3s16 start_padding)
*/
v3s16 last_room_center = roomplace + v3s16(roomsize.X / 2, 1, roomsize.Z / 2);

u32 room_count = random.range(dp.rooms_min, dp.rooms_max);
for (u32 i = 0; i < room_count; i++) {
for (u32 i = 0; i < dp.num_rooms; i++) {
// Make a room to the determined place
makeRoom(roomsize, roomplace);

Expand All @@ -219,7 +212,7 @@ void DungeonGen::makeDungeon(v3s16 start_padding)
#endif

// Quit if last room
if (i == room_count - 1)
if (i + 1 == dp.num_rooms)
break;

// Determine walker start position
Expand Down
38 changes: 19 additions & 19 deletions src/mapgen/dungeongen.h
View file
Expand Up @@ -46,32 +46,33 @@ struct DungeonParams {
content_t c_alt_wall;
content_t c_stair;

// Diagonal corridors are possible
bool diagonal_dirs;
// Dungeon only generates in ground
// 3D noise that determines which c_wall nodes are converted to c_alt_wall
NoiseParams np_alt_wall;

// Number of dungeons generated in mapchunk
u16 num_dungeons;
// Dungeons only generate in ground
bool only_in_ground;
// Number of rooms
u16 num_rooms;
// Room size random range. Includes walls / floor / ceilng
v3s16 room_size_min;
v3s16 room_size_max;
// Large room size
v3s16 room_size_large;
// First generated room is large
bool first_room_large;
// Dimensions of 3D 'brush' that creates corridors.
// Dimensions are of the empty space, not including walls / floor / ceilng.
v3s16 holesize;
// Corridor length
// Corridor length random range
u16 corridor_len_min;
u16 corridor_len_max;
// Room size includes walls / floor / ceilng
v3s16 room_size_min;
v3s16 room_size_max;
// The 1st room generated has a 1 in 4 chance of being 'large', all other
// rooms are not 'large'.
v3s16 room_size_large_min;
v3s16 room_size_large_max;
// Number of rooms
u16 rooms_min;
u16 rooms_max;
// Diagonal corridors are possible
bool diagonal_dirs;
// Usually 'GENNOTIFY_DUNGEON', but mapgen v6 uses 'GENNOTIFY_TEMPLE' for
// desert dungeons.
GenNotifyType notifytype;

// 3D noise that determines which c_wall nodes are converted to c_alt_wall
NoiseParams np_alt_wall;
};

class DungeonGen {
Expand All @@ -94,8 +95,7 @@ class DungeonGen {
DungeonGen(const NodeDefManager *ndef,
GenerateNotifier *gennotify, DungeonParams *dparams);

void generate(MMVManip *vm, u32 bseed,
v3s16 full_node_min, v3s16 full_node_max, u16 num_dungeons);
void generate(MMVManip *vm, u32 bseed, v3s16 full_node_min, v3s16 full_node_max);

void makeDungeon(v3s16 start_padding);
void makeRoom(v3s16 roomsize, v3s16 roomplace);
Expand Down
36 changes: 19 additions & 17 deletions src/mapgen/mapgen.cpp
View file
Expand Up @@ -870,29 +870,31 @@ void MapgenBasic::generateDungeons(s16 max_stone_y)
if (num_dungeons == 0)
return;

// Get biome at mapchunk midpoint
v3s16 chunk_mid = node_min + (node_max - node_min) / v3s16(2, 2, 2);
Biome *biome = (Biome *)biomegen->getBiomeAtPoint(chunk_mid);

PseudoRandom ps(blockseed + 70033);

DungeonParams dp;

dp.np_alt_wall =
NoiseParams(-0.4, 1.0, v3f(40.0, 40.0, 40.0), 32474, 6, 1.1, 2.0);

dp.seed = seed;
dp.num_dungeons = num_dungeons;
dp.only_in_ground = true;
dp.num_rooms = ps.range(2, 16);
dp.room_size_min = v3s16(6, 5, 6);
dp.room_size_max = v3s16(10, 6, 10);
dp.room_size_large = v3s16(
ps.range(10, 18), ps.range(8, 16), ps.range(10, 18));
dp.first_room_large = ps.range(1, 4) == 1;
dp.holesize = v3s16(2, 3, 2);
dp.corridor_len_min = 1;
dp.corridor_len_max = 13;
dp.rooms_min = 2;
dp.rooms_max = 16;
dp.diagonal_dirs = ps.range(1, 12) == 1;
dp.notifytype = GENNOTIFY_DUNGEON;

dp.np_alt_wall =
NoiseParams(-0.4, 1.0, v3f(40.0, 40.0, 40.0), 32474, 6, 1.1, 2.0);

dp.diagonal_dirs = false;
dp.holesize = v3s16(2, 3, 2);
dp.room_size_min = v3s16(6, 5, 6);
dp.room_size_max = v3s16(10, 6, 10);
dp.room_size_large_min = v3s16(10, 8, 10);
dp.room_size_large_max = v3s16(18, 16, 18);
dp.notifytype = GENNOTIFY_DUNGEON;
// Get biome at mapchunk midpoint
v3s16 chunk_mid = node_min + (node_max - node_min) / v3s16(2, 2, 2);
Biome *biome = (Biome *)biomegen->getBiomeAtPoint(chunk_mid);

// Use biome-defined dungeon nodes if defined
if (biome->c_dungeon != CONTENT_IGNORE) {
Expand All @@ -917,7 +919,7 @@ void MapgenBasic::generateDungeons(s16 max_stone_y)
}

DungeonGen dgen(ndef, &gennotify, &dp);
dgen.generate(vm, blockseed, full_node_min, full_node_max, num_dungeons);
dgen.generate(vm, blockseed, full_node_min, full_node_max);
}


Expand Down
17 changes: 10 additions & 7 deletions src/mapgen/mapgen_v6.cpp
View file
Expand Up @@ -568,14 +568,17 @@ void MapgenV6::makeChunk(BlockMakeData *data)
NoisePerlin3D(&np_dungeons, node_min.X, node_min.Y, node_min.Z, seed)), 0.0f);

if (num_dungeons >= 1) {
PseudoRandom ps(blockseed + 4713);

DungeonParams dp;

dp.seed = seed;
dp.num_dungeons = num_dungeons;
dp.only_in_ground = true;
dp.corridor_len_min = 1;
dp.corridor_len_max = 13;
dp.rooms_min = 2;
dp.rooms_max = 16;
dp.num_rooms = ps.range(2, 16);
dp.first_room_large = ps.range(1, 4) == 1;

dp.np_alt_wall
= NoiseParams(-0.4, 1.0, v3f(40.0, 40.0, 40.0), 32474, 6, 1.1, 2.0);
Expand All @@ -589,8 +592,8 @@ void MapgenV6::makeChunk(BlockMakeData *data)
dp.holesize = v3s16(2, 3, 2);
dp.room_size_min = v3s16(6, 9, 6);
dp.room_size_max = v3s16(10, 11, 10);
dp.room_size_large_min = v3s16(10, 13, 10);
dp.room_size_large_max = v3s16(18, 21, 18);
dp.room_size_large = v3s16(
ps.range(10, 18), ps.range(13, 21), ps.range(10, 18));
dp.notifytype = GENNOTIFY_TEMPLE;
} else {
dp.c_wall = c_cobble;
Expand All @@ -601,13 +604,13 @@ void MapgenV6::makeChunk(BlockMakeData *data)
dp.holesize = v3s16(1, 2, 1);
dp.room_size_min = v3s16(4, 4, 4);
dp.room_size_max = v3s16(8, 6, 8);
dp.room_size_large_min = v3s16(8, 8, 8);
dp.room_size_large_max = v3s16(16, 16, 16);
dp.room_size_large = v3s16(
ps.range(8, 16), ps.range(8, 16), ps.range(8, 16));
dp.notifytype = GENNOTIFY_DUNGEON;
}

DungeonGen dgen(ndef, &gennotify, &dp);
dgen.generate(vm, blockseed, full_node_min, full_node_max, num_dungeons);
dgen.generate(vm, blockseed, full_node_min, full_node_max);
}
}

Expand Down

0 comments on commit 0c31946

Please sign in to comment.