Closed laurelkeys closed 3 years ago
I'm just concerned about the fact that we trigger
on_rendering_abort
for both theAborted
andFailed
states.
on_rendering_abort
is used for both:
However, since the value of m_status
is set by do_render()
it'll either be Aborted
or Suceedded
(it does not return Failed
). Thus, the if (result.m_status == RenderingResult::Failed)
could simply be removed.
But, if do_render()
is later changed to return a Failed
state we won't be treating this case (or we could add it to the switch
statement, though it probably wouldn't make sense to call postprocess()
if the rendering failed).
I'm not sure, this if
statement was trying to be "future proof", so it may be better to remove it. What do you think?
Have you heavily tested appleseed.studio ? Also, be sure to open and try different projects when testing to find any corner case.
I have tested it with a couple of different combinations of post-processing stages (Color Map
, Render Stamp
, Vignette
) on quite a few images (small, large, non-square).
Also tried setting different color maps for False Colors
and everything appeared to be working š
Have you heavily tested appleseed.studio ? Also, be sure to open and try different projects when testing to find any corner case.
I have tested it with a couple of different combinations of post-processing stages (
Color Map
,Render Stamp
,Vignette
) on quite a few images (small, large, non-square).Also tried setting different color maps for
False Colors
and everything appeared to be working š
It's really important to test different scenes on top of the default "Cornell Box" scene.
The bug needs to be confirmed to continue to exist once #2877 is merged.
The bug needs to be confirmed to continue to exist once #2877 is merged.
It's still present after commit https://github.com/appleseedhq/appleseed/commit/0acfc054e9167ecfb25e3b5815ed0066e06b43f5.
PR https://github.com/appleseedhq/appleseed/pull/2896 adds the remaining changes to fix the issue, so I'm closing this one.
This PR fixes the following bug (which happens when we have post-processing stages, and
False Colors
is enabled):Edit: I undid the changes I mention below because they don't fit well with https://github.com/appleseedhq/appleseed/pull/2885 (though, I intend to add them in a later PR, once we discuss what'd lead to the best "user experience").
~Also, these changes make post-processing effects always run when a rendering is stopped (e.g.
Shift+F5
).~ ~If this behavior is desirable, https://github.com/appleseedhq/appleseed/pull/2878 can be closed. Otherwise, we can keeps effects only being applied to final renderings with:~