Exception noted when using bloom renderpass

[MAG-AdrianMeredith]

Description

I'm seeing an exception after testing the lastest builds when using the new renderpasses. Looking at the code in vscode makes it quite clear that it can be undefined so looks like a simple oversight

exception here
playcanvas.mjs:32653 Uncaught TypeError: Cannot read properties of null (reading 'id')
at ForwardRenderer.buildFrameGraph (playcanvas.mjs:32653:1)
at AppBase.renderComposition (playcanvas.mjs:42356:1)
at AppBase.render (playcanvas.mjs:42352:1)
at eval (playcanvas.mjs:42761:1)

I don't have an example repo case but hopefully the ide warning is enough

[mvaligursky]

Could you please look in the debugger to see when this happens? Render actions are created for rendering of layers, so typically this cannot be null. Is there maybe an instance of RenderPassForward with no layers added to it?

[MAG-AdrianMeredith]

Looks like it happens when we use our "Outline" posteffect it appears to be creating 3 renderActions, with the last one being totally empty. Any background information on what creates render actions?

[MAG-AdrianMeredith]

Do post effects still work in the new world or does this need porting to render passes?

[mvaligursky]

Post-effects work if you do not use render passes.
If you use render passes, you cannot use post-effects as those are not converted to render passes.

[MAG-AdrianMeredith]

Thats probably the issue then would it be unreasonable to log an error to the console in this case?

[mvaligursky]

Yeah that's definitely on the list. For now it's just a work in progress that's not released for public use. When we'll get it release-ready and documented, we'll port multiple engine examples to use it, make sure issues / errors are reported and similar.

I'll see if I can add some assets for case early though.

[MAG-AdrianMeredith]

Thanks for your help, this is is why I wanted to test things to see what breaks ;). I'll give it another go again next release or two