Python: Add missing argument

[sunyiming]

Motivation and Context

fix #5438

Description

Add missing argument in 05-using-the-planner.ipynb

Contribution Checklist

• The code builds clean without any errors or warnings
• The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
• All unit tests pass, and I have added new tests where possible
• I didn't break anyone 😄

[markwallace-microsoft]

Python 3.8 Test Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	5044	990	80%

report-only-changed-files is enabled. No files were changed during this commit :)

Python 3.8 Unit Test Overview

Tests	Skipped	Failures	Errors	Time
1059	11 💤	0 ❌	0 🔥	18.702s ⏱️

[markwallace-microsoft]

Python 3.9 Test Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	5181	984	81%

report-only-changed-files is enabled. No files were changed during this commit :)

Python 3.9 Unit Test Overview

Tests	Skipped	Failures	Errors	Time
1059	2 💤	0 ❌	0 🔥	21.640s ⏱️

[markwallace-microsoft]

Python 3.10 Test Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	5187	982	81%

report-only-changed-files is enabled. No files were changed during this commit :)

Python 3.10 Unit Test Overview

Tests	Skipped	Failures	Errors	Time
1059	1 💤	0 ❌	0 🔥	29.297s ⏱️

[markwallace-microsoft]

Python 3.12 Test Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	5187	982	81%

report-only-changed-files is enabled. No files were changed during this commit :)

Python 3.12 Unit Test Overview

Tests	Skipped	Failures	Errors	Time
1059	1 💤	0 ❌	0 🔥	30.523s ⏱️

[markwallace-microsoft]

Python 3.11 Test Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	5187	982	81%

report-only-changed-files is enabled. No files were changed during this commit :)

Python 3.11 Unit Test Overview

Tests	Skipped	Failures	Errors	Time
1059	1 💤	0 ❌	0 🔥	26.041s ⏱️

[moonbox3]

Hi @sunyiming, thanks for fixing this. Can you please run black formatting in the sk/python dir and push the changes so that step can pass? Thank you.

[sunyiming]

Hi @sunyiming, thanks for fixing this. Can you please run black formatting in the sk/python dir and push the changes so that step can pass? Thank you.

Hello. How to run black formatting? I tried to run poetry run black notebooks/05-using-the-planner.ipynb --check and got this:

Skipping .ipynb files as Jupyter dependencies are not installed.
You can fix this by running ``pip install "black[jupyter]"``
No Python files are present to be formatted. Nothing to do 😴

In this failed job https://github.com/microsoft/semantic-kernel/actions/runs/8249273901/job/22561304222?pr=5439 it looks like something wrong with dependencies installing.

[moonbox3]

Hi @sunyiming, thanks for fixing this. Can you please run black formatting in the sk/python dir and push the changes so that step can pass? Thank you.

Hello. How to run black formatting? I tried to run poetry run black notebooks/05-using-the-planner.ipynb --check and got this:

Skipping .ipynb files as Jupyter dependencies are not installed.
You can fix this by running ``pip install "black[jupyter]"``
No Python files are present to be formatted. Nothing to do 😴

In this failed job https://github.com/microsoft/semantic-kernel/actions/runs/8249273901/job/22561304222?pr=5439 it looks like something wrong with dependencies installing.

Please install the black package via pip (it exists outside of the poetry venv). You should install pip install black==24.2.0 so that you're using the same version that the GitHub actions uses. Then once installed (you may need to restart your shell), navigate to the semantic-kernel/python directory and run black .. That should show the file was updated with the proper formatting.

[sunyiming]

After running black .. under semantic-kernel/python, there is 215 files changed, 4765 insertions(+), 1500 deletions(-).
I think there must be something wrong. Is there any special configuration for black?

[moonbox3]

After running black .. under semantic-kernel/python, there is 215 files changed, 4765 insertions(+), 1500 deletions(-). I think there must be something wrong. Is there any special configuration for black?

You're running it like this?

[sunyiming]

@microsoft-github-policy-service agree