Conversation
- Makes sys path compatible with uv handling of paths.
- Short script that encapsulates adding the source directory to be used in jupyter notebooks (labs). - This allows to maintain DSI's folder structure and make it compatible with a "src" directory.
+ Simplified example + Enhanced visuals
+ Simplify example + Add reference to UCI ML repo
- Formatting - Wordsmith
+ General cleanup + Code tested under uv + Update path
+ Format + Spelling, clarity
+ Clean results + Verify everything runs
|
Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process. |
There was a problem hiding this comment.
Pull request overview
This pull request performs a comprehensive update and cleanup across multiple course materials, including notebooks, markdown slides, and source code. The changes focus on improving clarity, correcting spelling and grammar, updating data sources, simplifying examples, and verifying compatibility with the uv package manager.
Changes:
- General cleanup of slides and notebooks with improved spelling, grammar, and clarity
- Updated data sources and simplified examples in lab notebooks
- Added new dependencies (
seaborn,ucimlrepo) and helper scripts for path management - Enhanced logging and code structure in data processing pipelines
Reviewed changes
Copilot reviewed 20 out of 31 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Added seaborn and ucimlrepo dependencies |
| 05_src/stock_prices/data_manager.py | Enhanced logging and updated feature creation with explicit schema definitions |
| 03_instructional_team/markdown_slides/*.md | Corrected spelling/grammar and improved clarity throughout presentation slides |
| 01_materials/labs/update_path.py | New utility script for adding source directory to notebook paths |
| 01_materials/labs/*.ipynb | Updated notebooks with new data sources, improved examples, and consistent Python version references |
| 01_materials/labs/04_0_docker.md | New documentation file for Docker setup instructions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process. |
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
+N/A
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist