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

Overhaul anthill generation to avoid conflicting with other map features #36767

Merged
merged 1 commit into from
Jan 7, 2020

Conversation

kevingranade
Copy link
Member

@kevingranade kevingranade commented Jan 7, 2020

Summary

SUMMARY: None

Purpose of change

After the recent changes to keep anthills from conflicting with other terrain, #36736 #36702
Anthills would occasionally fail to generate due to colliding with other map features.
One failure was fatal since no tiles were placed.
Another resulted in labs overwriting pieces of anthills.
Fixes #27726

Describe the solution

Moved anthill generation after all other map features since they were conflicting with pre-cataclysm structures so anthills should never prevent construction of other structures.
Restructured anthill generation to not place features piecemeal.
Checked that the level under an anthill is clear before placing the anthill and associated resources.
Reset anthill if it is unable to be placed.

Testing

There was a fairly consistent failure that triggered in testing.
Also teleport across multiple overmaps and look for poorly generated anthills.

@kevingranade kevingranade added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing. Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Jan 7, 2020
@ZhilkinSerg ZhilkinSerg merged commit a68dc58 into CleverRaven:master Jan 7, 2020
@kevingranade kevingranade deleted the fix-anthils-again branch June 28, 2020 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing. Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ant Hill generation with no queen ant chamber
2 participants