-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AI Generative Readme And Sample Issue #33656
Comments
#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. |
Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Azure/azure-ml-sdk @azureml-github. |
1 similar comment
Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Azure/azure-ml-sdk @azureml-github. |
@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. |
@diondrapeck I agree with your point of view on #3. |
Thank you for the feedback, @jerryshia! For #1, it should link to creating an AI project. Do you agree? |
@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. |
@jerryshia @luigiw - the PR is ready. Can you both review + merge? |
#2 must have been fixed by someone else because and I just successfully imported it from there locally: |
@diondrapeck For issue #33968, LGTM. |
Ah yes, I misread your sentence. Thanks - I'll update it. |
Issue fixed. |
Section link1
Reason:
Link error.
Suggestion:
Update
Azure Machine Learning Workspace
toAzure OpenAI Service
. Add link "https://learn.microsoft.com/en-us/azure/ai-services/openai/overview "toAzure 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" toSamples
@rohit-ganguly , @lmazuel , @mayurid , @diondrapeck for notification.
The text was updated successfully, but these errors were encountered: