Skip to content

highforce material adjustments #570

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

Merged
merged 2 commits into from
Jul 22, 2025

Conversation

cjk28
Copy link
Contributor

@cjk28 cjk28 commented May 22, 2025

grass, trees, flowers, walls. tried to be as reserved as possible with the adjustments as there are many variables at play.

pure lcs (1) after
Screenshot_bmw_m3_e92_highforce_22-4-125-10-24-22
pure lcs (1) before
Screenshot_bmw_m3_e92_highforce_22-4-125-10-24-28
pure lcs (2) after
pure lcs (2) after
pure lcs (2) before
pure lcs (2) before
pure lcs (3) after
pure lcs (3) after
pure lcs (3) before
pure lcs (3) before
default wfx (1) after
default wfx (1) after
default wfx (1) before
default wfx (1) before
default wfx (2) after
default wfx (2) after
default wfx (2) before
default wfx (2) before

old values for reference:

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = grassBaked, terrain
PROP_... = ksAmbient, 0.2
PROP_... = ksDiffuse, 0.2

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = grassClip, grass
PROP_... = ksAmbient, 0.25
PROP_... = ksDiffuse, 0.25

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = walls
PROP_... = ksAmbient, 0.25
PROP_... = ksDiffuse, 0.25

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = Trees
PROP_... = ksAmbient, 0.2
PROP_... = ksDiffuse, 0.2

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = conifers
PROP_... = ksAmbient, 0.25
PROP_... = ksDiffuse, 0.25

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = flowersA
PROP_... = ksAmbient, 0.2
PROP_... = ksDiffuse, 0.2

old values for reference:

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = grassBaked, terrain
PROP_... = ksAmbient, 0.2
PROP_... = ksDiffuse, 0.2

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = grassClip, grass
PROP_... = ksAmbient, 0.25
PROP_... = ksDiffuse, 0.25

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = walls
PROP_... = ksAmbient, 0.25
PROP_... = ksDiffuse, 0.25

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = Trees
PROP_... = ksAmbient, 0.2
PROP_... = ksDiffuse, 0.2

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = conifers
PROP_... = ksAmbient, 0.25
PROP_... = ksDiffuse, 0.25

[SHADER_REPLACEMENT_...]
ACTIVE = 1
MATERIALS = flowersA
PROP_... = ksAmbient, 0.2
PROP_... = ksDiffuse, 0.2
@cjk28 cjk28 marked this pull request as ready for review May 22, 2025 09:25
@cjk28
Copy link
Contributor Author

cjk28 commented May 22, 2025

wanted to add @BrunUK as reviewer but not sure how

@Marocco2 Marocco2 requested a review from BrunUK May 22, 2025 10:39
@Marocco2
Copy link
Member

Marocco2 commented May 22, 2025

Changes do look good, although I'm wondering how they look at dawn/sunset as ksAmbient is higher than ksDiffuse (personally I much prefer the opposite)

In my experience, ksAmbient tends to glow too much in those scenarios and it'll pop up more with other surroundings, so I prefer reducing brightness by lowering ksAmbient rather than ksDiffuse

avoid higher ksambient than ksdiffuse except for grass, where lowering ksambient is causing grassfx to increase in brightness and look weird.
@cjk28
Copy link
Contributor Author

cjk28 commented May 22, 2025

yes makes sense, however lowering ksambient has basically no effect on the trees on this track for some reason. i've made another commit to at least make ksambient equal to ksdiffuse for everything apart from the grass tweak, as for some reason decreasing ksambient is causing grassfx to get much brighter and look weird. also didnt mess with the walls as they look balanced to me

heres a screenshot with/without my commits (with purelinear as pure filter seems to underexpose and crush blacks at dusk/dawn)

Screenshot_ks_mazda_miata_highforce_22-4-125-12-37-15
Screenshot_ks_mazda_miata_highforce_22-4-125-12-37-28

example of aforementioned grassfx issue if set ksambient to 0.16 to match ksdiffuse
Screenshot_ks_mazda_miata_highforce_22-4-125-12-40-45
vs 0.22 ksambient 0.16 ksdiffuse:
Screenshot_ks_mazda_miata_highforce_22-4-125-12-40-50

i also noticed that seasonal adjustments is working odd, and trees start to change to autumn as the sun goes down in summer, not sure if therse an easy fix. theres an ext config in the track as well as this one in the github maybe with some conflict

@RustyChest RustyChest merged commit f820df8 into ac-custom-shaders-patch:master Jul 22, 2025
@cjk28
Copy link
Contributor Author

cjk28 commented Jul 23, 2025

Brun wanted to review the changes before them being committed (but isn't using AC1 atm so may not happen), so maybe best to revert

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.

3 participants