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

Better scenario worldbuilding by giving PC knowledge of the nearest city on start #66781

Merged

Conversation

RenechCDDA
Copy link
Member

@RenechCDDA RenechCDDA commented Jul 9, 2023

Summary

Content "Most scenarios now provide vision of the nearest city on start"

Purpose of change

Some starts spawn outside of cities, but are not like... innawoods players detached from society. Presumably the PC had a home before the cataclysm, and some sort of education, and at least knew where the mcdonalds down the street was. The game doesn't currently represent that - it only plops down a big (15-tile default) radius of sight around you at the start, and if there's no civilization in sight, too bad, guess your character was living off the grid!

Describe the solution

Add a json toggle defined per-scenario (defaults true so it doesn't need to be added to 99% of starts) whether this vision should be granted.

Draw a path from the nearest road to the city center, revealing an area around it.

Reveal an area around the city center, scaling with the CITY_SIZE setting.

Describe alternatives you've considered

The current method of scaling with city size doesn't seem to be perfect? Sometimes cities are massive, sometimes cities are small, and the vision might either be insufficient or too much. But then again, I wouldn't expect a city's resident to know the entire city...

Testing

Here is an average evac shelter start:
image

Here is basically the worst-case scenario:
image

Additional context

Disabled this for the following scenarios:
Medieval peasant(DDA)
Lab patient(DDA)
Island Prison(DDA)
Mi-Go prisoner(DDA)
Portal dependent(DDA)
Aircraft carrier start(DDA)
Middle of nowhere(DDA)
Lost Underground(DDA)
All innawoods starts(just in case)
Wizard's Vacation(Magiclysm)
Exile (Magiclysm)

@RenechCDDA
Copy link
Member Author

@ Mod maintainers & those more versed in the scenarios than I am, I would be interested in hearing which scenarios might need to be updated to toggle this off?

@ashGlaw
Copy link
Contributor

ashGlaw commented Jul 9, 2023

Love this idea

@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. [C++] Changes (can be) made in C++. Previously named `Code` [Markdown] Markdown issues and PRs Scenarios New Scenarios, balancing, bugs with scenarios labels Jul 9, 2023
@GuardianDll
Copy link
Member

I wonder how it would work with alcatraz, as you can't draw a road here

which scenarios might need to be updated to toggle this off?

For magiclysm: lake_retreat, exile
For xedra: none

@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Jul 9, 2023
@RenechCDDA
Copy link
Member Author

If it can't find a road it actually draws nothing (silently fails).

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding Mods: Magiclysm Anything to do with the Magiclysm mod Mods: Innawood 🌲 Anything to do with Innawood mod BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jul 9, 2023
@github-actions github-actions bot requested a review from KorGgenT July 9, 2023 21:20
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-requesting reviews from non-collaborators: @GuardianDll @Light-Wave

@alp7292
Copy link
Contributor

alp7292 commented Jul 10, 2023

does this spawn world items. for example if you go to a unvisited place and load back save items and zombies will be different. but if you go to evac shelter and learn refugee center location everything on the path is generated and if you follow the road and load save back everything will be same. it most of the time generates places i will never visit and by generating it. it makes saving and loading times longer.

@RenechCDDA
Copy link
Member Author

No, it doesn't load any of the local maps.

image

@Inglonias
Copy link
Contributor

This is fantastic for someone who tends to play in and around cities like me. I love this.

@Maleclypse Maleclypse merged commit 3fc59ed into CleverRaven:master Jul 10, 2023
26 checks passed
@RenechCDDA RenechCDDA deleted the new_char_vision_of_nearest_city branch July 10, 2023 21:25
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 [C++] Changes (can be) made in C++. Previously named `Code` <Documentation> Design documents, internal info, guides and help. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions [Markdown] Markdown issues and PRs Mods: Innawood 🌲 Anything to do with Innawood mod Mods: Magiclysm Anything to do with the Magiclysm mod Mods Issues related to mods or modding Scenarios New Scenarios, balancing, bugs with scenarios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants