Map generation bug where some areas are unreachable about devilution HOT 11 CLOSED

Ok so the real issue was that the seed you provided is for level 2. Unlike Diablo 2 each dungeon level has it's own seed. In this case, the broken seed for level 4 is 708591219. We can test this like so:

Devilution.exe -l1 4 -r708591219 -w -d -^

from devilution.

I found another not reachable room in lvl2 with a current dev build (dba5e3d):

Savegame: single_0.zip

from devilution.

Fixed with #165. Tested all 3 broken seeds:

from devilution.

Would you please post the random seed? Press r in game. Also, create a save and upload if possible.

from devilution.

Would you please post the random seed? Press r in game. Also, create a save and upload if possible.

I just noticed that pressing r only works in debug builds.

@galaxyhaxz perhaps we should enable r to get the random seed also for the release version? At least until the majority of the bugs have been uncovered and fixed. The rest of the debug stuff can be in the debug specifc release.

from devilution.

galaxyhaxz perhaps we should enable r to get the random seed also for the release version? At least until the majority of the bugs have been uncovered and fixed. The rest of the debug stuff can be in the debug specifc release.

Definitely. Will make an update for that. We definitely need to know the map seed so we can trace through the DRLG code against the original and see what's messing up.

from devilution.

Seed: 699653937
Character: https://xyinn.org/diablo/debug/single_0.sv-2018-06-29-0845

Butcher already complete on this char (Obviously).

And yea, I second enabling debug by default in development builds. Also a new build should be released from HEAD.. 0.3?

from devilution.

I've tried the same map seed using builds from both Mingw and VC++ and the map generates correctly. Are you sure that's the right seed?

@mewmew Also, _DEBUG is now enabled by default, so all builds should contain them. The seed feature will come in handy.

from devilution.

@mewmew Also, _DEBUG is now enabled by default, so all builds should contain them. The seed feature will come in handy.

@galaxyhaxz Perfect! Another thing we may want to add is to output the git rev hash. Then we know for what game seed, and on what revision of Devilution.

from devilution.

@galaxyhaxz That's interesting, I didn't know every level had it's own seed.

from devilution.

I ran into the same issue on Dlvl 1 with the latest code.

from devilution.