Skip to content

Update/notebooks#164

Merged
calderonjesus merged 40 commits into
mainfrom
update/notebooks
Jun 14, 2026
Merged

Update/notebooks#164
calderonjesus merged 40 commits into
mainfrom
update/notebooks

Conversation

@calderonjesus

Copy link
Copy Markdown
Collaborator

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

This pull request primarily updates the introductory lab notebooks by cleaning up outputs, improving code clarity, and enhancing documentation formatting. It also removes the .python-version file, which specified the Python version for the environment.

Jupyter Notebook Cleanup and Improvements

  • Removed all cell outputs and set execution_count to null throughout 01_1_introduction.ipynb and 01_3_local_model.ipynb to provide a clean, ready-to-run notebook experience for learners. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15]

  • Updated notebook metadata to reflect Python version 3.12.7 and clarified the kernel display name. [1] [2]

Code and Documentation Enhancements

  • Improved code examples and documentation in 01_1_introduction.ipynb by:
    • Replacing commented code blocks with properly formatted markdown code blocks for better readability. [1] [2]
    • Refactoring OpenAI client instantiation logic to use environment variables for configuration, making the code more flexible and production-ready. [1] [2]

Environment Configuration

  • Removed the .python-version file, which previously pinned the Python version to 3.12. This change may impact environments that relied on this file for version management.

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?

N/A

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

N/A

How were these changes tested?

Ran all notebooks using the API Gateway and direct OpenAI API connection.

A reference to a related issue in your repository (if applicable)

N/A

Checklist

  • [X ] I can confirm that my changes are working as intended

@github-actions

Copy link
Copy Markdown

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.

@github-actions

Copy link
Copy Markdown

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.

@vishnouvina

Copy link
Copy Markdown
Collaborator

Do we want to keep the MCP content ?
I can see that we've gotten rid of the ngrok auth token and the mcp url, so 06_1_mcp.ipynb is probably outdated now.

@calderonjesus

Copy link
Copy Markdown
Collaborator Author

Do we want to keep the MCP content ? I can see that we've gotten rid of the ngrok auth token and the mcp url, so 06_1_mcp.ipynb is probably outdated now.

Thank you, @vishnouvina . Yes, we need to update the MCP portion of the repo. I opened Issue #165 to remind us of this item.

Comment thread 01_materials/labs/04_7_vectordb_docker.ipynb
Comment thread 01_materials/labs/02_4_eval_ai_judge.ipynb
@vishnouvina vishnouvina mentioned this pull request Jun 14, 2026
1 task
@calderonjesus calderonjesus merged commit f0ec703 into main Jun 14, 2026
1 check passed
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