Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
Co-authored-by: toyfer <15968477+toyfer@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR replaces the single polars package dependency with a curated set of nine pure-Python data analysis packages that don't require Rust toolchain compilation, making the embeddable Python distribution more compatible with environments lacking native build tools.
Changes:
- Updated dependency installation to include pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, and jupyterlab
- Updated release body metadata to reflect the new package list
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| .\Python\Python.exe -m pip install \ | ||
| pandas \ | ||
| numpy \ | ||
| matplotlib \ | ||
| seaborn \ | ||
| scikit-learn \ | ||
| openpyxl \ | ||
| xlrd \ | ||
| ipykernel \ | ||
| jupyterlab |
There was a problem hiding this comment.
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.
| .\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 |
| - **Python Version**: 3.13.7 | ||
| - **Package Manager**: pip | ||
| - **Added Package**: - | ||
| - **Added Package**: pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, jupyterlab |
There was a problem hiding this comment.
The label 'Added Package' is singular but lists multiple packages. Consider changing to 'Added Packages' for grammatical correctness.
| - **Added Package**: pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, jupyterlab | |
| - **Added Packages**: pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, jupyterlab |
| seaborn \ | ||
| scikit-learn \ | ||
| openpyxl \ | ||
| xlrd \ |
There was a problem hiding this comment.
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.
Swap out polars (requires Rust toolchain) for a set of pure-Python-compatible data-analysis packages suitable for embeddable distributions.
Changes
polarsinstall with 9 packages: pandas, numpy, matplotlib, seaborn, scikit-learn, openpyxl, xlrd, ipykernel, jupyterlabAll packages selected to avoid native compilation dependencies (no Rust/C++ toolchains needed).
Original prompt
This pull request was created from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.