Skip to content

Conversation

@Sooslick
Copy link

@Sooslick Sooslick commented Sep 9, 2023

Added "room2toilets" to required rooms list as far as room2toilets required for 100% walkthrough (by researcher james... achievement).
Fixed switch / case duplication for "room1archive" (making impossible to generate SCP-966 document)

@ChronoQuote
Copy link

+1 to these changes.

Forcing room2toilets to spawn is one of the main things I wanted to see in an update, and I saw your comment about it on #210, so I'm not sure why I didn't make a PR sooner lol.

The position min_pos+Floor(0.1*Float(Room2Amount[2])) is already given to room2poffices on line 7431, so even though this change uses SetRoom to try giving that position to room2toilets, SetRoom will actually place room2toilets somewhere else. I had thought of putting room2toilets at min_pos+Floor(0.25*Float(Room2Amount[2])) instead, right below room2cafeteria. Either way the bathroom will tend to spawn "near" the cafeteria which I think makes sense.

It's also worth noting that adding rooms to these forced-spawn lists lowers the spawn likelihood of the rooms later in the list, since in the event that not enough rooms of that particular room type are available, the rooms first in the list get priority. Luckily it seems the last three rooms on this particular forced-spawn list aren't associated with any achievements (although some do have documents no other rooms have, but eventually I'd like to submit an improvement to SCP-1162's random document selection so that all-document runs are feasible even when certain rooms don't spawn).

@Sooslick
Copy link
Author

Sooslick commented Sep 11, 2023

the bathroom will tend to spawn "near" the cafeteria which I think makes sense.

Yep. Moved this to 0.25.

not enough rooms of that particular room type are available

As the solution for "all documents run" I suggest shift EZ boundary by one or two cells forward. So, EZ will have higher chance to generate 2 horisontal lines to fill them with these rooms.

@Regalis11 Regalis11 merged commit ded4312 into Regalis11:dev Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants