Skip to content

Replace polars with curated data-analysis packages in embeddable Python workflow #3

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.

Sign up for GitHub

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

Merged
toyfer merged 2 commits into from
Jan 16, 2026
+11 −2
Merged

Replace polars with curated data-analysis packages in embeddable Python workflow #3

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
480eb13
Initial plan
Copilot Jan 16, 2026
cf7afee
Update workflow to install curated data-analysis packages
Copilot Jan 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 11 additions & 2 deletions .github/workflows/init-python.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,16 @@ jobs:

- name: Python Embeddable Setup Dependencies
run: |
.\Python\Python.exe -m pip install polars
.\Python\Python.exe -m pip install \
pandas \
numpy \
matplotlib \
seaborn \
scikit-learn \
openpyxl \
xlrd \
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The package xlrd is primarily used for reading old Excel files (.xls format). Since openpyxl handles modern Excel formats (.xlsx), consider documenting why xlrd is needed, or evaluate if it's necessary given that xlrd development has been discontinued and only supports .xls files up to Excel 2003 format.

Copilot uses AI. Check for mistakes.
ipykernel \
jupyterlab
Comment on lines +32 to +41
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Installing multiple packages in separate lines can be slower than installing them all in a single command. While PowerShell line continuation with backslash works, this approach may result in multiple pip resolution passes. However, since this appears to be the intended formatting per the PR description, this is informational rather than a required change.

Suggested change
.\Python\Python.exe -m pip install \
pandas \
numpy \
matplotlib \
seaborn \
scikit-learn \
openpyxl \
xlrd \
ipykernel \
jupyterlab
.\Python\Python.exe -m pip install pandas numpy matplotlib seaborn scikit-learn openpyxl xlrd ipykernel jupyterlab

Copilot uses AI. Check for mistakes.

- name: Compress Python
run: Compress-Archive -Path .\Python\* -DestinationPath .\Python_Embeddable.zip
Expand All @@ -44,7 +53,7 @@ jobs:

- **Python Version**: 3.13.7
- **Package Manager**: pip
- **Added Package**: -
- **Added Package**: pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, jupyterlab
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The label 'Added Package' is singular but lists multiple packages. Consider changing to 'Added Packages' for grammatical correctness.

Suggested change
- **Added Package**: pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, jupyterlab
- **Added Packages**: pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, jupyterlab

Copilot uses AI. Check for mistakes.
- **Build Date**: ${{ github.event.head_commit.timestamp }}
- **Commit**: ${{ github.sha }}

Expand Down
Loading