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

Fix arm64 devcontainer build by updating container base image, install useful vscode extensions by default #120

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

Conversation

katyapotapov
Copy link

When I tried to run this devcontainer on a MacBook M1 Pro, the primp library failed to build because compatible wheels were not available, and the build process requires Cargo (the Rust package manager) which is not installed in the container. As seen in the primp README, Linux aarch64 wheels are only compatible with Debian >=12 whereas the devcontainer base image was using Debian 11.

This change upgrades the base image to use Debian 12 for compatibility with existing primp wheels, and also upgrades to devcontainer 1.0 and Python 3.11. Also added some extensions that should be installed anyways to work with Jupyter notebooks inside the devcontainer (I think ms-toolsai.jupyter, ms-python.python and ms-azuretools.vscode-docker are required to run the notebooks, and ms-python.vscode-pylance adds code completion and other nice features).

Testing these changes: Successfully built and ran the devcontainer and the first notebook on my local machine and in GH Codespaces. I can leave another comment on this PR once I run the rest of the notebooks to ensure there are no issues.

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.

1 participant