Skip to content

chore: move Main.DrawSettings -> `.HumanModel.DrawSettings - #180

Open
bkje14 wants to merge 32 commits into
ammr4-betafrom
drawsettings-refactor
Open

chore: move Main.DrawSettings -> `.HumanModel.DrawSettings#180
bkje14 wants to merge 32 commits into
ammr4-betafrom
drawsettings-refactor

Conversation

@bkje14

@bkje14 bkje14 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

The BodyModel had a lot of absolute paths to the default DrawSettings. This and the location of the drawsettings directly inside Main creates issues when wanting to include two humans in a model. Furthermore, Drawsettings are primarily for the HumanModel.

This PR moves the default settings to HumanModel.DrawSettings.
The old location is marked as deprecated and can be switched on using the BM_COMPATIBILITY_BODYMODEL_STRUCTURE parameter.

The two BM parameters BM_DRAWSETTINGS_FILE and BM_DRAWSETTINGS_SKIP as they are no longer needed.
The default drawsettings are implemented with the ??= optional operator (since AMMR2).
The BM_DRAWSETTINGS_SKIP was a temporary workaround to allow two humans in one model.

@github-actions github-actions Bot added the chore label Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant