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

Update Docker file #2899

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

david-cortes-intel
Copy link
Contributor

@david-cortes-intel david-cortes-intel commented Sep 16, 2024

Description

This PR makes some adjustments to the Docker file provided under /dev:

  • Downloads Miniforge (hosted on GitHub) instead of Miniconda (hosted on Anaconda).
  • Replaces usage of conda with the newer mamba. Did not work in the CI.
  • Pulls everything from mamba from the conda-forge repository.
  • Uses the latest version of the instructions for adding the intel APT repository, which requires some additional packages.
  • Changes the fixed version of dpcpp towards the last version instead, as it comes from a non-ubuntu repository which is subject to changes and might at some point no longer host the specific version that was added here.
  • Adds the runtime libraries for opencl instead of manually adding a text file entry in its place.
  • Adds some short instructions for building the container locally.

I'm not 100% sure that the resulting image will end up building (haven't figured out my networking settings for Docker yet), but hopefully the CI jobs here should bring some light.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Issue/PR related to oneDAL docs infra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant