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

with 2 or more groups of lights + certain AOVs fails #336

Closed
Noah-ok opened this issue Apr 14, 2020 · 2 comments
Closed

with 2 or more groups of lights + certain AOVs fails #336

Noah-ok opened this issue Apr 14, 2020 · 2 comments
Assignees
Labels

Comments

@Noah-ok
Copy link

Noah-ok commented Apr 14, 2020

when I try to render with a group of lights plus all the AOVs activated I get this
2020-04-13-220009

deactivating these AOVs works
2020-04-13-220658

LuxCoreRender 2.4 Blender 2.82a

untitled1.zip

@Dade916
Copy link
Member

Dade916 commented Apr 14, 2020

@Theverat this should be consider (somewhat) a BlendLuxCore bug: LuxCore does a lexical sort of imagepipelines names because they are not required do be numbers. So the following order of image pipelines will be defined in the attached test scene:

film.imagepipelines.0
film.imagepipelines.1
film.imagepipelines.10
film.imagepipelines.11
film.imagepipelines.12
film.imagepipelines.13
film.imagepipelines.2
film.imagepipelines.3
film.imagepipelines.4
film.imagepipelines.5
film.imagepipelines.6
film.imagepipelines.7
film.imagepipelines.8
film.imagepipelines.9

And the definition of image pipeline "10" after "1" will trigger the error (i.e. "1" name should be "01" instead).

I modified the LuxCore code so it try to sort using numerical order if the image pipeline names can be converted to a number. So it now works fine even with the above order. It should be more intuitive.

@Theverat
Copy link
Member

Ok, I have addressed this in LuxCoreRender/BlendLuxCore@5b3744e

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

No branches or pull requests

3 participants