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

[Xedra Evolved] Replace solid fog field with solid fog terrain #75191

Merged
merged 3 commits into from
Jul 25, 2024

Conversation

Standing-Storm
Copy link
Contributor

@Standing-Storm Standing-Storm commented Jul 23, 2024

Summary

Mods "[Xedra Evolved] Replace solid fog field with solid fog terrain"

Purpose of change

The sylph altar was causing massive slowdown for anyone near it, presumably due to trying to calculate light levels or sight distances or something similar through the fog field.

I didn't notice this when I first PRed it because of debug clairvoyance. You didn't change, you didn't improve, nothing was ventured and nothing was gained etc.

Describe the solution

Change the field into terrain.

If some version of #61872 gets PRed, the fog can be translucent.

Describe alternatives you've considered

Testing

It works, no slowdown.

Additional context

@Standing-Storm Standing-Storm marked this pull request as draft July 23, 2024 16:37
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding Map / Mapgen Overmap, Mapgen, Map extras, Map display Mods: Xedra Evolved Anything to do with Xedra Evolved labels Jul 23, 2024
@github-actions github-actions bot requested a review from Maleclypse July 23, 2024 16:38
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Jul 23, 2024
@Standing-Storm Standing-Storm marked this pull request as ready for review July 24, 2024 01:24
@github-actions github-actions bot added BasicBuildPassed This PR builds correctly, label assigned by github actions and removed BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jul 24, 2024
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jul 25, 2024
@Maleclypse Maleclypse merged commit 75f28b3 into CleverRaven:master Jul 25, 2024
24 checks passed
@Standing-Storm Standing-Storm deleted the solid-fog-wall branch July 25, 2024 16:38
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 Mods: Xedra Evolved Anything to do with Xedra Evolved Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants