Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move refugee center start from isolationist to its own scenario #79140

Merged
merged 3 commits into from
Jan 13, 2025

Conversation

further-reading
Copy link
Contributor

@further-reading further-reading commented Jan 13, 2025

Summary

Content "Move refugee center start from isolationist to its own scenario"

Purpose of change

The refugee center is a very interesting location as it opens up many of the features/content that makes this game stand out: mainly the faction system (through recruiting quest givers found at this location), the lore (by speaking to the NPCs present) and its sci-fi content (as some NPCs might tell you where NPC factions like the exodii can be found). However, it generally spawns very far away in my experience, making the trek out there feel like a daunting prospect. I wanted to alleviate this by offering a scenario to start right at the center.

Describe the solution

I noticed that the "Isolationist" scenario already had a Refugee center start location, but I felt it didn't make sense under that banner since starting in a room full of people is hardly Isolationist, so I moved it out to its own scenario. Additionally, I changed the terrain to spawn the player outside the entrance rather than inside a dorm because I felt it flows better if you're entering the center for the first time at the beginning of your run.

Describe alternatives you've considered

I considered leaving the terrain untouched so you'd start in a dorm rather than on the road. However the narrative that you're arriving at the start of the run fits better.

I considered starting the player closer to the door- in the garden outside the door to be specific. However I kept starting behind trees or surrounded by bushes and it felt a little janky.

Testing

I started 5 new games and checked if I started on the road path outside the front entrance each time.

Additional context

None.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Scenarios New Scenarios, balancing, bugs with scenarios labels Jan 13, 2025
@Procyonae
Copy link
Contributor

Procyonae commented Jan 13, 2025

I noticed that the "Isolationist" scenario already had a Refugee center start location, but I felt it didn't make sense under that banner since starting in a room full of people is hardly Isolationist

That's only the id it's called Safe Place in game which isn't that egregious (not that they let you stay) I agree that it makes more sense as its own scenario though and spawning you outside when there's no special handling to start you "as part of" the refugee center or something makes more sense. Also given the amount of peeps that seem to be annoyed when they spawn a "long" way from the refugee center, having an obvious scenario that's just "we skipped the drive for you" seems good.

@github-actions github-actions bot added Map / Mapgen Overmap, Mapgen, Map extras, Map display json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jan 13, 2025
@GuardianDll GuardianDll merged commit 566eb1c into CleverRaven:master Jan 13, 2025
26 of 29 checks passed
@further-reading further-reading deleted the refugee_center_start branch January 14, 2025 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display new contributor Scenarios New Scenarios, balancing, bugs with scenarios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants