azure-search-openai-demo icon indicating copy to clipboard operation
azure-search-openai-demo copied to clipboard

Update README.md

Open msdpalam opened this issue 2 years ago • 1 comments

Purpose

  • ... Found that without the py launcher in the path, the azd up is failing. I feel that we need to add a bit more clarification on py launcher dependency for azd up, so adding the following: "Important - Please note that azd up would look for py launcher, make sure py launcher is in the path system environment variable. After successful installation of Python (with py launcher selected in the installer), put the py location (i.e., C:\Windows) in the system path variable.

Please review and let me know if we can add this change. It took me some time to figure this out, hopefully will save some end user time seeing the same behavior. One of my customers had this issue. Even though I had Python in path, azd up kept failing and only way to fix this was to put the py launcher location in the path system variable.

Does this introduce a breaking change?

[ ] Yes
[ x] No

Pull Request Type

What kind of change does this Pull Request introduce? Documentation

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ x] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code No code changes

  • Test the code

What to Check

Verify that the following are valid

  • ... Review the information added in the README.md and test if needed. Please reach out, if I can help with the clarification

Other Information

msdpalam avatar Apr 19 '23 16:04 msdpalam

I don't have the py launcher and was able to run this template successfully, how did you install Python? Python for Windows from the Windows store and python.org comes bundled with the Py launcher.

tonybaloney avatar Jun 29 '23 04:06 tonybaloney