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

Improve Rendering Script #17

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

lalalune
Copy link

@lalalune lalalune commented Oct 8, 2022

This PR aims to improve the data generation pipeline in the following ways:

  • Add significant documentation
  • Add flag to switch from Cycles renderer to Eevee renderer, Cycles is still default
  • Add flag to adjust num_views
  • Move all scene setup, lighting and other repeat functional into a Blender scene which is loaded instead -- one for Cycles, one for Eevee -- this makes debugging and modifying scene settings much more interactive
  • Switch lighting scheme from massive area light to primarily HDR image-based lighting
  • Move dataset selection to an overridable JSON file, which allows for more fine-grained path control and custom datasets
  • Retain same defaults as before

Please let me know if this works for your needs or if you'd like to change / reduce scope on anything!

Please note, if you accept the general pattern of this PR I would like to give the code a once over and make sure everything is really perfect before it goes into the codebase. Thanks!

@lalalune lalalune marked this pull request as ready for review October 8, 2022 09:41
@TREE-Ind
Copy link

TREE-Ind commented Oct 9, 2022

2022-10-09 10_34_48-paperspace@psb8uby3q_ ~_GET3D_render_shapenet_data

Nice work, we were able to get ShapenetV2 rendering with Blender 3.3 using your PR :D

@muxizju
Copy link

muxizju commented Oct 10, 2022

@lalalune I get errors when using this pr, it seems the blender can not recognize the config file "eevee_renderer.blend".
my blender version is 2.9.0, is it not matched ?
image

@lalalune
Copy link
Author

@muxizju I saved the project in the most recently stable Blender download (3.3?) so it might be a version conflict there

@SteveJunGao SteveJunGao mentioned this pull request Oct 11, 2022
@muxizju
Copy link

muxizju commented Oct 13, 2022

@muxizju I saved the project in the most recently stable Blender download (3.3?) so it might be a version conflict there

that's right, use version 3.3 can solve it.
While it seems the EEVEE engine can not work on terminal-only linux, there is an error“Unable to open a display
Aborted (core dumped)”. It works well with CYCLES on the same machine

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

Successfully merging this pull request may close these issues.

4 participants