maaz.code logo
Simple view

Mergedmatplotlib/matplotlib#3237523.4Kstars

FIX: Update devcontainer base image to resolve Storage limitations.

By Mohd Maaz Khan, merged by story645 into matplotlib/matplotlib on 22 Sept 2026

+8−88 lines added, 8 lines removed2 files changed4 comments

Conversation

  1. maaz7409opened this pull request on 19 Sept 2026description.mdAuthor

    PR summary

    This PR updates the mcr.microsoft.com/devcontainers/universal:2 image to base:ubuntu image. It also makes following changes to ensure container is light, flexible and supports setup.sh installation :

    • Changes strict requirement of 4-cores setup to flexible option for 2-cores or 4-cores setup.

    • Adds SSH feature.

    • Adds Conda ( since base:ubuntu doesn't readily comes with Conda ).

    • Updates conda environment path for base:ubuntu.

    • Explicitly invokes bash for onCreateCommand to avoid permission related issues.

    This only consumes 8 GB space. I have tested this and it works fine.

    IMG_20260919_172746 IMG_20260919_172941

    Fixes #32374

    AI Disclosure

    No AI was used.

    PR quality check

    • Use an expressive title, e.g. "Fix title font property precedence"
    • [N/A] New and changed code is tested
    • [N/A] Plotting related features are demonstrated in an example
    • [N/A] New features and API changes have release notes
    • [N/A] Documentation complies with general and docstring guidelines
  2. Maharshi-Paulreviewed on 20 Sept 2026

    Maharshi-Paulcommented on 20 Sept 2026

    Was dropping hostRequirements entirely intentional or should cpus/memory be explicitly (e.g cpus: 2) so Codespaces doesnt default to an unpredictable size for a build this heavy?

  3. maaz7409commented on 20 Sept 2026Author

    @Maharshi-Paul it was intentional, because that lets you choose between 2-cores cpu and 4-cores cpu. ( When creating it by selecting New with Options ). IMG_20260920_204550

    And when you create directly without setting any options, Codespace defaults to 2-cores cpu in this case. However, explicitly defining it to be 2-core cpu will not let you choose 4-core cpu for heavy work and vice-versa. Hence dropping it was the flexible option.

  4. melissawmapproved these changes on 21 Sept 2026

    melissawmcommented on 21 Sept 2026Member

    Does look reasonable and works fine, thank you!

  5. story645approved these changes on 22 Sept 2026

  6. story645commented on 22 Sept 2026Member

    Thanks for fixing this!

  7. story645merged this pull request on 22 Sept 2026

Copied from GitHub when this site was built; comments by bots are left out.