Skip to content

Conversation

@mmcky
Copy link
Contributor

@mmcky mmcky commented Nov 20, 2025

Summary

This PR updates the CI workflow to use the new ghcr.io/quantecon/quantecon:latest container instead of manually setting up the environment.

Changes

  • ✅ Use quantecon container in ci.yml
  • ✅ Remove manual setup-miniconda and LaTeX installation steps
  • ✅ Simplify to single conda env update command
  • ✅ Remove redundant test-container.yml workflow

Benefits

  • Faster builds: Pre-installed Python 3.13, Anaconda 2025.06, LaTeX
  • More reliable: Consistent environment across all builds
  • Simpler maintenance: ~40% fewer lines in workflow file
  • Better caching: Container layers cached by Docker

Testing

Container successfully built and pushed to GHCR:

This PR will trigger the updated ci.yml workflow to validate the container works correctly.

- Compare container build vs ubuntu-latest baseline
- Validate environment setup and package installation
- Generate build artifact comparison
- Measure performance improvements
- Replace setup-miniconda + manual LaTeX install with container
- Remove redundant test-container.yml workflow
- Simplify to single conda env update step
- Remove shell: bash -l {0} (not needed in container)
@netlify
Copy link

netlify bot commented Nov 20, 2025

Deploy Preview for jade-tarsier-d98a19 ready!

Name Link
🔨 Latest commit 681c9a3
🔍 Latest deploy log https://app.netlify.com/projects/jade-tarsier-d98a19/deploys/691e9691a0629d0008aae6fe
😎 Deploy Preview https://deploy-preview-9--jade-tarsier-d98a19.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

- Add --pull always option to container config
- Ensures latest ghcr.io/quantecon/quantecon:latest is used
- Bypasses GitHub Actions runner cache
- GitHub Actions already pulls latest tag when changed
- Force-pull adds 2min overhead unnecessarily
- Previous issue was timing (test before container built)
- All required packages already in container base image
- environment.yml duplicated container contents (python=3.13, anaconda=2025.06, jupyter-book)
- Embracing full container approach for cleaner workflow
- Saves 30-60s per build
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.

2 participants