-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
Blender 2.8 [$430 awarded] #166
Comments
Seems like there are some regressions in the RenderEngine API: https://devtalk.blender.org/t/2-8-render-engine-api-papercuts/6878 |
Short status update: What works
Todo
Thanks to all the donors for their support! I appreciate it a lot. |
A first testbuild is available here: https://github.com/LuxCoreRender/BlendLuxCore/releases/tag/blendluxcore_v2.2beta3 |
With the release of BlendLuxCore v2.2 for both Blender 2.79 and 2.80 I consider this issue solved. Starting now, Blender 2.79 will no longer be supported by future versions of the addon and all development efforts will concentrate on Blender 2.80. |
This issue is used to track the progress of the Blender 2.8 port of LuxCore's Blender addon.
You can contribute to the development by raising the bounty on this issue.
Info about the required changes:
Todo:
Preparation (still 2.79 compatible):
__init__.py
files (so auto_load.py can find them)bpy.types
classes into multi inheritance (e.g. if we have base classLuxCoreNode
, then subclasses likeLuxCoreNodeMatte
should use multi inheritance:LuxCoreNodeMatte(bpy.types.Node, LuxCoreNode)
)bpy.utils.register_module
(deprecated in 2.80)2.80 Port
Basics:
template_ID()
function from the Blender API for our material UI because we need not only a custom "new" operator, but also a custom "copy" operator (otherwise the node tree is not copied and two different materials share the same node tree). So we need to build our own UI that looks as close to thetemplate_ID()
result as possible (see the 2.79 BlendLuxCore code for an example)depsgraph.scene
ordepsgraph.scene_eval
(documentation is missing, read the Blender source code or ask in devtalk) -> Cycles usesscene_eval
, so we should do the samecontext.region.view2d.view_to_region
?)depsgraph.scene
instead ofdepsgraph.scene_eval
in render engine for halt conditions)Extras:
The $430 bounty on this issue has been claimed at Bountysource.
The text was updated successfully, but these errors were encountered: