microsoft / semantic-kernel

Integrate cutting-edge LLM technology quickly and easily into your apps
https://aka.ms/semantic-kernel
MIT License
21.26k stars 3.12k forks source link

Python: Bug: Broken link in tutorial - semantic-kernel/python/samples/getting_started/05-using-the-planner.ipynb #7284

Open SeryioGonzalez opened 1 month ago

SeryioGonzalez commented 1 month ago

Describe the bug One step in the tutorial points to a broken link

To Reproduce Steps to reproduce the behavior: See the block:

The sequential planner is an XML-based step-by-step planner. You can see the prompt used for it here (https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planning/sequential_planner/Plugins/SequentialPlanning/skprompt.txt)

Expected behavior Update the link:

(https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planners/sequential_planner/Plugins/SequentialPlanning/skprompt.txt)

See the path planning has been renamed to planners in the code, but not in the link above

moonbox3 commented 1 month ago

@SeryioGonzalez, thanks for filing. Would you like to create a PR to fix it?

SeryioGonzalez commented 1 month ago

Sure! I actually tried pushing you a PR, but realize had no permits

moonbox3 commented 1 month ago

Sure! I actually tried pushing you a PR, but realize had no permits

The PR must be generated from a fork of the repo.