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

Craters are spawning partially #73950

Closed
Brambor opened this issue May 20, 2024 · 4 comments
Closed

Craters are spawning partially #73950

Brambor opened this issue May 20, 2024 · 4 comments
Labels
Character / World Generation Issues and enhancements concerning stages of creating a character or a world (S3 - Duplicate) Bug that is duplicate of another one

Comments

@Brambor
Copy link
Contributor

Brambor commented May 20, 2024

Describe the bug

See screenshots.

Attach save file

N/A

Steps to reproduce

  1. Make a world
  2. Run around and find craters.

Expected behavior

Circular or ellipsoid or some sensibly shaped craters.

Screenshots

image

image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19045.4412 (22H2)
  • Game Version: 0.G-9614-gf522d7df2d-dirty [64-bit] - master was 39e2afa
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

No response

@Brambor Brambor added (S1 - Need confirmation) Report waiting on confirmation of reproducibility Character / World Generation Issues and enhancements concerning stages of creating a character or a world labels May 20, 2024
@NetSysFire
Copy link
Member

This sounds like it is related to #73435 and the PR that fixes it.

@PatrikLundell
Copy link
Contributor

I don't think it's caused by that PR, because I've seen it in the past.

My guess would be that the explosion that's triggered when the "bomb" enters the reality bubble only affects the part of the OMT that's inside the reality bubble. If so, you should be able to work around it by having the explosion handling load the OMT in a smallmap (you need to affect at least the surface level and the level below that, with deeper craters a possible future enhancement), apply the explosion there, and then proceed. I think this would also have the positive side effect of allowing you to hide the explosion (it's supposed to have happened a long time ago), although you'd still affect local wildlife at the current time.
Note that you'd need to have different handling for the case of crater generation from the one for"real" explosions that happen well within the confines of the reality bubble, as you definitely want your hand grenades to produce effects.

@GuardianDll
Copy link
Member

Duplicate of #59781

The most easiest solution is to place bomb in the center of OMT. This will work for medium-power bombs (such as on crater map extra), with explosion radius not crossing the boundaries of OMT. Consequently, this solution won't work for high-power bombs, as even one bomb with enough destructive power can have very big explosion radius which will cross the boundaries of OMT.

The proper solution would be to rework explosion mechanic so it can handle crossing the boundaries of OMT.

@GuardianDll GuardianDll closed this as not planned Won't fix, can't repro, duplicate, stale May 20, 2024
@PatrikLundell
Copy link
Contributor

If explosions are to be able to cross OMT boundaries then I'd suggest the use of a map centered on the center submap in a manner similar to how we use tinymaps (and fakemaps, and smallmaps) to generate stuff. Let the explosion occur on this map as soon as the reality bubble touches the outermost submap of the explosion radius, or the OMT or submap of the explosion center, of dealing with radii is too hard.
Instead of a map you could use something smaller but still large enough, but I suspect a map is good enough.
The side effect of that implementation is that OMTs around the explosion would be generated a little earlier than normal, but that really only affects save scumming for the generation (or non generation) of stuff on those OMTs.

@Brambor Brambor added (S3 - Duplicate) Bug that is duplicate of another one and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Character / World Generation Issues and enhancements concerning stages of creating a character or a world (S3 - Duplicate) Bug that is duplicate of another one
Projects
None yet
Development

No branches or pull requests

4 participants