Skip to content
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

Add enterprise file search #47614

Merged
merged 4 commits into from
Dec 19, 2024
Merged

Add enterprise file search #47614

merged 4 commits into from
Dec 19, 2024

Conversation

nick863
Copy link
Contributor

@nick863 nick863 commented Dec 19, 2024

  • Add enterprise file search sample
  • Add Code interpreter sample
  • Add batch file search and enterprise batch file search sample
  • Add code changes to expose appropriste interfaces.

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

Azure.AI.Projects

@dargilco dargilco requested a review from Copilot December 19, 2024 15:04
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 5 out of 13 changed files in this pull request and generated 4 comments.

Files not reviewed (8)
  • sdk/ai/Azure.AI.Projects/api/Azure.AI.Projects.net8.0.cs: Evaluated as low risk
  • sdk/ai/Azure.AI.Projects/tests/Samples/Agent/Sample_Agent_Enterptise_File_Search.cs: Evaluated as low risk
  • sdk/ai/Azure.AI.Projects/api/Azure.AI.Projects.netstandard2.0.cs: Evaluated as low risk
  • sdk/ai/Azure.AI.Projects/src/Custom/Agent/FileSearchToolResource.cs: Evaluated as low risk
  • sdk/ai/Azure.AI.Projects/src/Custom/Agent/MessageAttachment.cs: Evaluated as low risk
  • sdk/ai/Azure.AI.Projects/tests/AIProjectsTestEnvironment.cs: Evaluated as low risk
  • sdk/ai/Azure.AI.Projects/tests/Samples/Agent/Sample_Agent_OpenAPI.cs: Evaluated as low risk
  • sdk/ai/Azure.AI.Projects/tests/Samples/Agent/Sample_Agent_Vector_Store_Batch_Enterptise_File_Search.cs: Evaluated as low risk
Comments suppressed due to low confidence (2)

sdk/ai/Azure.AI.Projects/tests/Samples/Agent/Sample_Agents_Code_Interpreter_File_Attachment.cs:44

  • Incorrect list initialization syntax. Use 'new List()' instead of square brackets.
var attachment = new MessageAttachment(fileId: fileId, tools: tools);

sdk/ai/Azure.AI.Projects/tests/Samples/Agent/Sample_Agents_Code_Interpreter_File_Attachment.cs:14

  • [nitpick] Class name should be 'SampleAgentsCodeInterpreterFileAttachment' for consistency.
public partial class Sample_Agents_Code_Interpreter_File_Attachment : SamplesBase<AIProjectsTestEnvironment>

@nick863 nick863 merged commit e3602f9 into main Dec 19, 2024
17 checks passed
@nick863 nick863 deleted the nirovins/add_pnterprise_file_search branch December 19, 2024 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants