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

Issues with Custom Autotiler Features #435

Open
coloursofnoise opened this issue Apr 15, 2022 · 0 comments
Open

Issues with Custom Autotiler Features #435

coloursofnoise opened this issue Apr 15, 2022 · 0 comments
Labels
bug Something isn't working rewrite An existing feature needs a rewrite

Comments

@coloursofnoise
Copy link
Member

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: coloursofnoise@f9e5582

@coloursofnoise coloursofnoise added bug Something isn't working rewrite An existing feature needs a rewrite labels Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working rewrite An existing feature needs a rewrite
Projects
None yet
Development

No branches or pull requests

1 participant