Skip to content

Conversation

@jetrotal
Copy link
Contributor

@jetrotal jetrotal commented Dec 7, 2025

I guess this completes the last feature from Rewrite Map Commap.

Introduces IsWaterTile and IsDeepWaterTile helpers and implements specialized autotiling logic for water and deep water tiles in RecalculateAutotile. This enables more accurate coastline and deep water boundary rendering, including diagonal and cardinal neighbor checks for seamless water transitions.


image image

@jetrotal jetrotal force-pushed the maniacs/rewrite_map_final branch from f6909c7 to 8c52c59 Compare December 8, 2025 02:49
@jetrotal
Copy link
Contributor Author

jetrotal commented Dec 8, 2025

solved the latest issue found, thanks to @fdelapena for pointing out info about out of bounds tiles

image

@jetrotal jetrotal force-pushed the maniacs/rewrite_map_final branch 2 times, most recently from bf4c854 to 8c52c59 Compare December 8, 2025 04:10
Copy link
Contributor

@fdelapena fdelapena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. It could be nice adding comments or function names on parts are from patches or maniac patch specific imo.

Introduces IsWaterTile and IsDeepWaterTile helpers and implements specialized autotiling logic for water and deep water tiles in RecalculateAutotile. This enables more accurate coastline and deep water boundary rendering, including diagonal and cardinal neighbor checks for seamless water transitions.
@jetrotal jetrotal force-pushed the maniacs/rewrite_map_final branch from 8c52c59 to de9b5dd Compare December 13, 2025 02:17
Enables the 'replace range' feature in CommandManiacRewriteMap by using a lambda to handle tile ID resolution and iteration. Now supports replacing a range of tiles using variable values, improving map editing flexibility.
@jetrotal
Copy link
Contributor Author

There was a missing feature: Range Mode. Implemented that and tested in my end. Results on pair with maniacs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants