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

Wyrms mine jsonify #51167

Merged
merged 7 commits into from
Aug 30, 2021
Merged

Wyrms mine jsonify #51167

merged 7 commits into from
Aug 30, 2021

Conversation

Night-Pryanik
Copy link
Contributor

@Night-Pryanik Night-Pryanik commented Aug 28, 2021

Summary

Content "Wyrms mine jsonify"

Purpose of change

Continuation of ongoing project of dehardcoding mine mapgen.

Closes #13310.
Closes #48288.

Describe the solution

  • Added orifice terrain for dark wyrms to crawl from. Spawned fields of acid near them as dark wyrms produce acid trails when moving.
  • Made wyrm pedestal sealed, so players won't be able to take petrified eye by g or AIM anymore. They could get the eye only through examine.
  • Made "wyrms" tunnels snaky in accordance with the "snakes/wyrms" theme.
  • The whole location has only 2 underground levels versus 3 levels for "amigara" or "spiral" versions.
  • Overall number of wyrms should be higher than in previous, hardcoded, version, but my testing showed that wyrms are not that scary as opponents, and I was able to easily walk away from them (didn't even run). The bigger problem might be their acid trails as now tunnels are much more narrower than before.

Describe alternatives you've considered

None.

Testing

Debug-spawned mine, teleported to the finale. Examined the pedestal, took the eye. Didn't want to fight with wyrms, successfully walked away from them. Checked if everything is ok.

Additional context

Snaky tunnels with wyrms' orifices and puddles of acid near them. Wyrms will get out only when the eye will be removed from the pedestal
изображение

Finale
изображение

@Night-Pryanik Night-Pryanik added the Map / Mapgen Overmap, Mapgen, Map extras, Map display label Aug 28, 2021
@Theawesomeboophis
Copy link
Contributor

Great Job, this is a long-running bug that I'm glad we can finally squash.

@kevingranade kevingranade merged commit 85a9936 into CleverRaven:master Aug 30, 2021
@Night-Pryanik Night-Pryanik deleted the wyrms-mine-jsonify branch August 30, 2021 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
4 participants