crowdrender / cr-docker

The official repository for the CrowdRender docker image
59 stars 8 forks source link

Docker file changes, shell script changes #21

Closed Jeducious closed 8 months ago

Jeducious commented 1 year ago

Some Big Changes!

Motivation

The main motivation here is to make the docker system more flexible, this means allowing users to build images themselves using whatever base image and version of Blender they wish. It also means allowing for using development branches of the addon inside a running container via a bind mount. The really helps include docker images in our development cycle as currently the only way to test if new code in the addon works in a container is to publish the addon live, and then pull it into a running container.

Changes

Dockerfile

Shell scripts

Other Chnages

Jeducious commented 8 months ago

Hi @zocker-160, I've just pushed commits to this branch/PR. Would really appreciate if you can let me know what you think and if we can merge. I really would like to get these containers back in use, since we've not updated them in ages, I think people are less likely to use them since Blender has moved on and is about to move to V4.0.

I think the changes I've made will help, I've tested GPU support on Docker Desktop in windows with a new docker file that is based on the original. All works well.

Cheers!

zocker-160 commented 8 months ago

Seems to be working from my short testing, merging now after our conversation on Slack, but more changes will be needed (especially in the Readme which is kinda a mess IMO).