EverestAPI / EverestAPI/Everest

Issues with Custom Autotiler Features

Open
#435 0 comments 0 reactions 0 assignees View on GitHub
bug rewrite
Dominant language
C#
Stars
516
Forks
106
Avg merge
4d 9h
Merged PRs (30d)
5

Description

The original implementation is inherently flawed because padding and fill is applied after only checking if there are tiles in all surrounding positions, which completely disregards custom definitions or "not this tile" masks.

Tileset masks are also incorrectly sorted in priority, which does not match the vanilla implementation.

Expanding the scan size for padding and fill levels to match a custom scan width/height should also be properly trialed to determine if it actually has a notable impact on performance.

At a minimum, a vanilla tileset parsed using the custom parser and handled by the custom autotiler should behave correctly, which it currently does not.

A very unfinished attempt at fixing some of these issues: https://github.com/coloursofnoise/Everest/commit/f9e55826fbc3cad5a354af509d91be00606b0c44

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.