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

Automapping ignore empty outputs #3896

Merged
merged 9 commits into from
Feb 26, 2024

Conversation

bjorn
Copy link
Member

@bjorn bjorn commented Feb 26, 2024

This change introduces a "compileOutputSet" step which is executed as part of collecting the rules. In this step, each rule only receives as possible outputs the output indexes that are non-empty for the region of that rule.

This makes it much easier to add output variation to only some of the rules. Previously, due to the possibility of empty outputs being chosen, having rules with various amounts of variation required setting up separate rule maps.

Closes #3523

This PR supercedes #3894, which was merged accidentally.

This change introduces a "compileOutputSet" step which is executed as
part of collecting the rules. In this step, each rule only receives as
possible outputs the output indexes that are non-empty for the region of
that rule.

This makes it much easier to add output variation to only some of the
rules. Previously, due to the possibility of empty outputs being chosen,
having rules with various amounts of variation required setting up
separate rule maps.

Closes mapeditor#3523
Even if it doesn't actually place any tiles.
It did not have QKeyValueIterator::operator-> yet.
By dragging the target layer name around we avoid the need for a lookup
map later on. Doesn't reduce line count, though, and likely uses slightly
more memory.
'myst' cross-reference target not found: 'object-properties' [myst.xref_missing]
@bjorn bjorn merged commit c691061 into mapeditor:master Feb 26, 2024
12 of 13 checks passed
@bjorn bjorn deleted the automapping-ignore-empty-outputs branch February 26, 2024 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automapping: Ignore empty outputs per-rule
1 participant