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

[BUG]: INITIAZLIZATION WITHOUT GPU #723

Closed
muedavid opened this issue Oct 10, 2022 · 4 comments
Closed

[BUG]: INITIAZLIZATION WITHOUT GPU #723

muedavid opened this issue Oct 10, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@muedavid
Copy link
Contributor

Describe the bug
bproc.init() calls RendererUtility.set_render_devices(). However in set_render_devices fails if no GPU is connected. There is a flag to set use_cpu. However this flag can not be set in the init() function. Thus currently blenderproc can only be used with GPU.

My suggestion would be to either allow to set the flag to the init() function itself or if it fails to find a GPU device to use CPU per default.

General Information

  1. Which BlenderProc version are you using? 2.5

  2. On which operating system are you? Ubuntu

  3. Have you checked the issue tracker to see if a similar issue has been opened? Yes

  4. Have you changed BlenderProc in any way besides the config file? If yes, are you sure that this change does not affect the problem you are having? Yes

To Reproduce
Steps to reproduce the behavior:

  1. just run any blenderproc file without having a GPU
@muedavid muedavid added the bug Something isn't working label Oct 10, 2022
@themasterlink
Copy link
Contributor

Hey,

yes, it seems that here:

# Make sure desired_gpu_device_type is a list
if desired_gpu_ids is not None and not isinstance(desired_gpu_ids, list):
desired_gpu_ids = [desired_gpu_ids]
# Decide between gpu and cpu rendering
if not desired_gpu_device_type or use_only_cpu:

No check is performed to validate that these GPU types exist.

Best,
Max

@muedavid
Copy link
Contributor Author

Hi Max

Thanks for your reply. Should I send you a PR for this small change or are you going to update it ?

Best,

David

@themasterlink
Copy link
Contributor

That hasn't been fixed yet. I will keep this open until it does, ok? ;)

@themasterlink themasterlink reopened this Oct 19, 2022
@MartinSmeyer
Copy link
Member

This was actually solved in #740

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants