Skip to content

Navigation Menu

Sign in
Sign up

feat: HBAO + SSGI + TRAA + MotionBlur - #211

Open
CodyJasonBennett wants to merge 13 commits into
pmndrs:master from
CodyJasonBennett:feat/ssgi
Open

feat: HBAO + SSGI + TRAA + MotionBlur #211
CodyJasonBennett wants to merge 13 commits into
pmndrs:master from
CodyJasonBennett:feat/ssgi

Conversation

@CodyJasonBennett

@CodyJasonBennett CodyJasonBennett commented May 23, 2023
edited
Loading

Copy link
Copy Markdown
Contributor

Implements #183
Implements 0beqz/realism-effects#5

Follows up #210 by adding HBAO + SSGI + TRAA + MotionBlur passes from 0beqz/realism-effects.

<EffectComposer disableNormalPass multisampling={0}>
 <HBAO /> // Note: this is a component of SSGI and thus redundant
 <SSGI />
 <TRAA />
 <MotionBlur />
</EffectComposer>

https://codesandbox.io/s/c143do

image

vis-prime, v1rtl, LeviBroadnax, and eatdesignlove reacted with heart emoji
Comment thread src/effects/SSGI.tsx Outdated
Comment thread src/effects/SSGI.tsx Outdated
@CodyJasonBennett CodyJasonBennett changed the title (ε‰Šι™€) feat: SSGI (ε‰Šι™€γ“γ“γΎγ§) (追記) feat: SSGI + TRAA (θΏ½θ¨˜γ“γ“γΎγ§) May 24, 2023
@CodyJasonBennett CodyJasonBennett changed the title (ε‰Šι™€) feat: SSGI + TRAA (ε‰Šι™€γ“γ“γΎγ§) (追記) feat: SSGI + TRAA + MotionBlur (θΏ½θ¨˜γ“γ“γΎγ§) May 24, 2023
@CodyJasonBennett CodyJasonBennett changed the title (ε‰Šι™€) feat: SSGI + TRAA + MotionBlur (ε‰Šι™€γ“γ“γΎγ§) (追記) feat: HBAO + SSGI + TRAA + MotionBlur (θΏ½θ¨˜γ“γ“γΎγ§) May 24, 2023

v1rtl commented May 27, 2023

Copy link
Copy Markdown
Member

@CodyJasonBennett there's merge conflicts, also safe to merge after?

CodyJasonBennett commented May 27, 2023
edited
Loading

Copy link
Copy Markdown
Contributor Author

Can clear those in a moment. Need to do integration testing still, but it could be worthwhile to wait until 0beqz updates SSGI as it dramatically reduces memory usage and smearing. Need to also see if there's a diffuse-only or specular-only (SSR) GI and whether we can combine his effects at all. I get a broken render when combining all into an effectpass as per his readme.

dswhy commented Jun 21, 2023

Copy link
Copy Markdown

Any estimation on when this could be available? I would really love to use the motion blur :D

tiivik, dfeehrer, eatdesignlove, dongho-shin, KihwanChoi12, and kitaedesigns reacted with thumbs up emoji

Copy link
Copy Markdown

Wow I cannot wait for this merge! 🀀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@drcmda drcmda drcmda left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle γ«γ‚ˆγ£γ¦ε€‰ζ›γ•γ‚ŒγŸγƒšγƒΌγ‚Έ (->γ‚ͺγƒͺγ‚ΈγƒŠγƒ«) /