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

Volumetrics: Blend from last frame to avoid ugly banding in viewport #9

Open
wants to merge 1 commit into
base: bonemaster292
Choose a base branch
from

Conversation

lordloki
Copy link
Contributor

@lordloki lordloki commented Jan 5, 2021

Como comentaste en el video que hiciste ayer sobre Blender VR que tenías problemas con el banding de las volumétricas en el viewport, te paso este Pull request que te añade una opción para que las volumetricas de cada frame hagan un blend al 95% con el anterior frame y de esta manera desaparezca el banding.

El patch no añade mucho código. Te paso un video donde se ve mas o menos como queda. El video es del patch sobre UPBGE pero se ve bastante bien aunque los controles no son los mismos (en este patch el botón se llama volumetric Blending y está en la zona de volumétricas): https://www.youtube.com/watch?v=onCc3O4P_S8&feature=youtu.be

This follows Frostbite's paper (and EEVEE F12 render) for blending over
95% of last frame.
@juangea
Copy link
Owner

juangea commented Apr 5, 2021

Thanks a lot, sorry for the late answer, I'll generate a new branch for Eevee modifications and apply this one :)

I usually generate separated branches for different features, so I have BoneCycles, BoneCloth and we used to have BoneopenVDB, so right now for this I will create BoneEevee and I can apply the patch there, this is to have every modification under control and if something needs update we can build bonemaster without it until it's up to date :)

@lordloki
Copy link
Contributor Author

lordloki commented Apr 6, 2021

Yw :-). If anytime you need help updating the patch, just ping me.

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.

2 participants