AI Generative Readme And Sample Issue

[jerryshia]

Section link1

Reason:
Link error.

Suggestion:
Update Azure Machine Learning Workspace to Azure OpenAI Service. Add link "https://learn.microsoft.com/en-us/azure/ai-services/openai/overview "to Azure OpenAI Service

Section link1

Reason:
'module' object is not callable

Suggestion:
Update code to from azure.ai.generative.evaluate import evaluate

Section link1

Reason:
Link not included.

Suggestion:
Update code to Samples ,Add link"https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/ai/azure-ai-generative/samples" to Samples

@rohit-ganguly , @lmazuel , @mayurid , @diondrapeck for notification.

[diondrapeck]

#33515 was already in progress to address 3

I'll make PRs to address 1 and 2 and close this Issue once they're merged.

[github-actions]

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Azure/azure-ml-sdk @azureml-github.

[github-actions]

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Azure/azure-ml-sdk @azureml-github.

[diondrapeck]

@jerryshia - #3 is actually not a bug. That samples folder you linked to is the internal folder we use in the codebase to host in-line code examples that are shown along with reference documentation. That's not what we want users to go to directly to learn how to use the product.

We did have a typo in that Samples link which was addressed in #33515, but the solution was to redirect it to azureai-samples which is our dedicated Code Samples repository with e2e examples of different scenarios to use the product with.

I'll be addressing #1 in PR #33968 and #2 in a later PR.

[jerryshia]

@diondrapeck I agree with your point of view on #3.
About #1 ,the PR #33968 is not we expected. The package in #33968 is not the same as the target package we want to modify. The package we intend to change is the azure-ai-generative ,while the package modified in the PR is the azure-ai-ml. Please follow my suggestion to do the update.

[luigiw]

Thank you for the feedback, @jerryshia! For #1, it should link to creating an AI project. Do you agree?

[diondrapeck]

@jerryshia - #33968 is still in progress. I'm fixing a batch of reference docs bugs across our 3 SDKs in that PR, so sorry for the confusion but that azure-ai-ml change is intentional. I will add the changes we've discussed here for the AI SDKs today.

[diondrapeck]

@jerryshia @luigiw - the PR is ready. Can you both review + merge?

[diondrapeck]

#2 must have been fixed by someone else because evaluate() is now exported in the __init__.py file for the evaluate module: https://github.com/Azure/azure-sdk-for-python/blob/27316b3a56ae9181e050afa2df2381d5872b54a0/sdk/ai/azure-ai-generative/azure/ai/generative/evaluate/__init__.py#L16C1-L17C1

and I just successfully imported it from there locally:

[jerryshia]

@diondrapeck For issue #33968, LGTM.
About #2, yes, the package azure.ai.generative.evaluate can be successfully imported, but the package in the sample file is azure.ai.generative. So we want to update azure.ai.generative to azure.ai.generative.evaluate

[diondrapeck]

Ah yes, I misread your sentence. Thanks - I'll update it.

[jerryshia]

Issue fixed.