Skip to content

Docker patch#22

Merged
jonasteuwen merged 7 commits intomainfrom
docker_patch
Jul 17, 2021
Merged

Docker patch#22
jonasteuwen merged 7 commits intomainfrom
docker_patch

Conversation

@frank
Copy link
Contributor

@frank frank commented Jul 12, 2021

Raises versions of python, pytorch, cuda, ubuntu. Also adds the jupyter notebook config back.

@frank frank requested a review from jonasteuwen July 12, 2021 07:45
@frank frank self-assigned this Jul 12, 2021
@frank
Copy link
Contributor Author

frank commented Jul 12, 2021

I just found that while this builds, the conda environment solver is installing torchvision 0.2.2 which is an old version. Let's hold on merging this until I figure out how to fix that

@frank
Copy link
Contributor Author

frank commented Jul 12, 2021

The torchvision issue is solved with the latest commit. Python 3.9 was the issue

@frank
Copy link
Contributor Author

frank commented Jul 13, 2021

Looks like the pixman bug is re-introduced with this patch to the Dockerfile. Hold merging until that's fixed

@frank
Copy link
Contributor Author

frank commented Jul 13, 2021

Seems like the bug doesn't happen with ubuntu 18, so downgrading did the trick. All is in order now

@jonasteuwen
Copy link
Contributor

jonasteuwen commented Jul 13, 2021 via email

@frank
Copy link
Contributor Author

frank commented Jul 13, 2021

I haven't been able to do that in docker yet, the meson --prefix=/usr line in the commands I reported in issue#20 fails (with meson installed ofc)

- Make resulting docker smaller
- Install pixman and cairo from scratch
- Compile pytorch and torchvision specifically for dlup
@jonasteuwen jonasteuwen merged commit 647ccdf into main Jul 17, 2021
@jonasteuwen jonasteuwen deleted the docker_patch branch July 17, 2021 16:06
@frank frank self-assigned this Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants