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

Rare shader output is previous frame. #7

Open
plebghudmgfujygthmjdmfgjsdrgjsdrt opened this issue Feb 9, 2025 · 7 comments
Open

Rare shader output is previous frame. #7

plebghudmgfujygthmjdmfgjsdrgjsdrt opened this issue Feb 9, 2025 · 7 comments
Assignees
Labels
question Further information is requested

Comments

@plebghudmgfujygthmjdmfgjsdrgjsdrt

Occassionaly the shader will output the previous output image it has been passed not the current one.

Something going on with the frame buffer?

eg:-
a goes in modified a is output
b goes in previous a output is output
c goes in modified c is output

the output is unmodified so the shader isn't reusing the input image, it is bypassing the code and outputting the last frames output.

Happens ~1% of the time

most of the time this isn't much of a problem as you can just generate again, but with long sequences...

https://www.youtube.com/watch?v=axBK9pEOMLQ

it is stopping and starting occasionally because of this.

@plebghudmgfujygthmjdmfgjsdrgjsdrt
Copy link
Author

video is 16x slo-mo btw.

@Amorano Amorano self-assigned this Feb 9, 2025
@Amorano Amorano added the question Further information is requested label Feb 9, 2025
@Amorano
Copy link
Owner

Amorano commented Feb 9, 2025

Cool video =D

I think I would need to see the workflow (some cut out version) to figure this out -- or maybe some more information on how many frames? you are generating? Hard to trace without more information.

Technically there is no hold out for the buffer, so, if its getting its "previous" frame that could simply be the ComfyUI execution engine which caches outputs if the inputs don't change -- I do "force" the output, even if the inputs "don't change" but that is probably where I would start to trace this.

I will take more of a look tomorrow.

@plebghudmgfujygthmjdmfgjsdrgjsdrt
Copy link
Author

@plebghudmgfujygthmjdmfgjsdrgjsdrt
Copy link
Author

plebghudmgfujygthmjdmfgjsdrgjsdrt commented Feb 10, 2025

Image
Image
Image
Image
Image
Image

email kept wigging out about file size so here are the example images, they should all be a stereo pair.

@Amorano
Copy link
Owner

Amorano commented Feb 10, 2025

You will also need (though you could bypass it by giving the depth input the diffuse), the lotus depth nodes and model. If comfy doesn't auto download the model...

I think I missed something. I dont need a model -- I need a workflow. If the model itself is the problem, then that is the model's problem.

I need a workflow that makes the "skips" otherwise I cant count that as a problem for my nodes?

Was there some other part of the message that got clipped?

@plebghudmgfujygthmjdmfgjsdrgjsdrt
Copy link
Author

@plebghudmgfujygthmjdmfgjsdrgjsdrt
Copy link
Author

BugTest.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants