Skip to content

Commit

Permalink
Merge pull request #1048 from solliancenet/cp-bugfix-e2e
Browse files Browse the repository at this point in the history
Bug fixes for vectorization e2e tests
  • Loading branch information
ciprianjichici authored May 24, 2024
2 parents 44239f8 + 05c2edb commit 03c5fd0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
8 changes: 4 additions & 4 deletions tests/dotnet/Core.Examples/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -170,8 +170,8 @@ The vectorization api and vectorization job managed identities need to have `Con
##### App Config settings
| Key | Value | Description |
| --- | --- | --- |
| `FoundationaLLM:DataSources:datalake_vectorization_input:AuthenticationType` | `AzureIdentity` | The authentication method for the vectorization api and vectorization job managed identities. This will always be `AzureIdentity`. |
| `FoundationaLLM:DataSources:datalake_vectorization_input:AccountName` | `onelake` | Account name - this will always be `onelake`. |
| `FoundationaLLM:DataSources:onelake_fllm:AuthenticationType` | `AzureIdentity` | The authentication method for the vectorization api and vectorization job managed identities. This will always be `AzureIdentity`. |
| `FoundationaLLM:DataSources:onelake_fllm:AccountName` | `onelake` | Account name - this will always be `onelake`. |

#### Running the example

Expand Down Expand Up @@ -215,8 +215,8 @@ The vectorization api and vectorization job managed identities need to have `Con
##### App Config settings
| Key | Value | Description |
| --- | --- | --- |
| `FoundationaLLM:DataSources:datalake_vectorization_input:AuthenticationType` | `AzureIdentity` | The authentication method for the vectorization api and vectorization job managed identities. This will always be `AzureIdentity`. |
| `FoundationaLLM:DataSources:datalake_vectorization_input:AccountName` | `onelake` | Account name - this will always be `onelake`. |
| `FoundationaLLM:DataSources:onelake_fllm:AuthenticationType` | `AzureIdentity` | The authentication method for the vectorization api and vectorization job managed identities. This will always be `AzureIdentity`. |
| `FoundationaLLM:DataSources:onelake_fllm:AccountName` | `onelake` | Account name - this will always be `onelake`. |

#### Running the example

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -341,18 +341,19 @@ await coreClient.PostAsync($"instances/{instanceId}/providers/{resourceProvider}

async public Task<IndexingProfile> GetIndexingProfile(string name)
{
return await GetResourcesAsync< IndexingProfile>(instanceSettings.Value.Id, ResourceProviderNames.FoundationaLLM_Vectorization, $"indexingProfiles/{name}");

return (await GetResourcesAsync<ResourceProviderGetResult<IndexingProfile>>(instanceSettings.Value.Id, ResourceProviderNames.FoundationaLLM_Vectorization, $"indexingProfiles/{name}")).Resource;

}

async public Task<TextEmbeddingProfile> GetTextEmbeddingProfile(string name)
{
return await GetResourcesAsync< TextEmbeddingProfile>(instanceSettings.Value.Id, ResourceProviderNames.FoundationaLLM_Vectorization, $"textEmbeddingProfiles/{name}");
return (await GetResourcesAsync<ResourceProviderGetResult<TextEmbeddingProfile>>(instanceSettings.Value.Id, ResourceProviderNames.FoundationaLLM_Vectorization, $"textEmbeddingProfiles/{name}")).Resource;
}

async public Task<TextPartitioningProfile> GetTextPartitioningProfile(string name)
{
return await GetResourcesAsync<TextPartitioningProfile>(instanceSettings.Value.Id, ResourceProviderNames.FoundationaLLM_Vectorization, $"textPartitioningProfiles/{name}");
return (await GetResourcesAsync<ResourceProviderGetResult<TextPartitioningProfile>>(instanceSettings.Value.Id, ResourceProviderNames.FoundationaLLM_Vectorization, $"textPartitioningProfiles/{name}")).Resource;
}
}
}

0 comments on commit 03c5fd0

Please sign in to comment.