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

Rivers don't connect smoothly across overmap boundaries #36630

Open
sayke opened this issue Jan 2, 2020 · 7 comments
Open

Rivers don't connect smoothly across overmap boundaries #36630

sayke opened this issue Jan 2, 2020 · 7 comments
Labels
<Bug> This needs to be fixed Map / Mapgen Overmap, Mapgen, Map extras, Map display

Comments

@sayke
Copy link
Contributor

sayke commented Jan 2, 2020

Describe the bug

Rivers, riverbanks, lakes, and roads don't seem to properly connect to each other across overmap boundaries.

Steps To Reproduce

Steps to reproduce:

  1. Create a new character in a new world.
  2. Use debug console to reveal map (overmap).
  3. Use debug console to teleport to where a river crosses an overmap boundary.
  4. Observe weirdly-connected river and riverbank tiles.

Expected behavior

I would expect rivers, lakes, and roads to connect smoothly across overmap boundaries, rather than forming sharp lines along them.

Screenshots

These screenshots illustrate the problem both on the local map and on the overmap.
Screen Shot 2020-01-01 at 11 42 13 PM
Screen Shot 2020-01-01 at 11 43 25 PM

Versions and configuration

  • OS: MacOs
  • Game Version: 0.D-10932-g9eb7975 [64-bit]
  • Graphics version: Tiles (UndeadPeople)
  • Mods loaded: [dda, no_npc_food, No_Rail_Stations, sleepdeprivation, StatsThroughSkills]

Additional context

Rivers, lakes, and road connections all seem to get weird across overmap tiles, so it looks like this may have something to do with the build_connection code that gets used during overmap generation...?

@ZhilkinSerg
Copy link
Contributor

That looks like map extra placed over invalid overmap terrain.

@ZhilkinSerg ZhilkinSerg added <Bug> This needs to be fixed Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Jan 2, 2020
@sayke
Copy link
Contributor Author

sayke commented Jan 2, 2020

It might look like that in those screenshots, but check this out - could a map extra result in such a smooth line cutting off the lake across the overmap boundary like this? I'm pretty sure it's happening right across the overmap boundary here:
Screen Shot 2020-01-01 at 9 34 03 PM

@Night-Pryanik
Copy link
Contributor

Seems like resurrection of #18583?

@sayke
Copy link
Contributor Author

sayke commented Jan 2, 2020

Seems like resurrection of #18583?

This seems different in that it only shows up along overmap boundaries, but it has to be at least somewhat related. I'm just not familiar enough with the map generation process to figure out how connection-building (I suspect that may be involved?) is supposed to interface with overmap creation.

@ralreegorganon
Copy link
Contributor

Lakes do connect smoothly across overmap boundaries, by design. In all of the screenshots here, all of the waterbodies are rivers. Rivers absolutely do have boundary problems.

@Night-Pryanik Night-Pryanik changed the title Rivers/lakes/roads don't connect smoothly across overmap boundaries Rivers/roads don't connect smoothly across overmap boundaries Jan 10, 2020
@Diabolus-Engi
Copy link

Do roads have this issue as well then ? Or just the rivers ?

@ralreegorganon
Copy link
Contributor

Just rivers. Roads will attempt to continue their connections across overmap boundaries, and ever since #33949 and #33996 other linear features like subways and sewers can do so as well (though no code currently pushes them up to the boundaries such that this would be triggered). Roads have a separate problem with nonsensical connectivity between locations, but that is a different issue.

@ralreegorganon ralreegorganon changed the title Rivers/roads don't connect smoothly across overmap boundaries Rivers don't connect smoothly across overmap boundaries Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants