andreluizsecco / cognitive-luis-programmatic Goto Github PK

Get All Intents returning only 100 intents. How to get all the intents?

The Base URL in ServiceClient.cs is hardcoded and useless to Europe/Australia

Hi,
I wanted to ask: if I use an Intent getter like GetIntentByNameAsync(...) and the intent selected does not exists the method returns null? Or it tries to create a new Intent and returns that?

Thanks

Implement the feature that creates a new version equivalent to the current snapshot of the selected application version.
https://westus.dev.cognitive.microsoft.com/docs/services/5890b47c39e2bb17b84a55ff/operations/5890b47c39e2bb052c5b9c3a

I'm thinking about changing the namespaces (in the next version) and structures of the project to organize and facilitate a bit more at the time of use.
As resources increase, the use of the same namespace becomes more confusing.

ps: obviously, this will cause a breaking change.

Example

Now:

using (var client = new LuisProgClient("Your_Authoring_Key", Regions.WestUS))
{
    var apps = await client.GetAllAppsAsync();

    var intents = await client.GetAllIntentsAsync(
        appId: "24d2a592-165d-468f-8800-7778900afc11",
        appVersionId: "0.1"
    );
}

After:

using (var client = new LuisProgClient("Your_Authoring_Key", Regions.WestUS))
{
    var apps = await client.Apps.GetAllAsync();
    
    var intents = await client.Intents.GetAllAsync(
        appId: "24d2a592-165d-468f-8800-7778900afc11",
        appVersionId: "0.1"
    );
}

I would like to know your opinions and if anyone has any objections.

We currently have a problem trying to run all the tests at the same time, having to run the test classes separately.
It is necessary to make improvements in the unit tests to allow the execution of all the tests at the same time.

Implement the feature that updates the application publish settings.
https://westus.dev.cognitive.microsoft.com/docs/services/5890b47c39e2bb17b84a55ff/operations/5ae32174d5b81c02cc64c6d3

Dear @andreluizsecco,
I want to let you know I'm working on supporting the /apps/{appId}/versions endpoint.

This is needed to cover my use case:
My use case is, to export the whole app (/apps/{appId}/versions/{versionId}/export) to json and then import it again on another luis instance (used for DevOps / CI/CD).

I'm working on it here:
https://github.com/michel-weber/Cognitive-LUIS-Programmatic/tree/feature/versionservice

I'm also implementing/extending your integration tests to cover my new code and trying to stay close to your naming/coding conventions.

Please let me know if you have any guidance on contributing to your project.

Kind regards
Michel

Implement the feature that updates the application settings.
https://westus.dev.cognitive.microsoft.com/docs/services/5890b47c39e2bb17b84a55ff/operations/58aeface39e2bb03dcd5909e

Hello Andre, it's possible to use Cognitive-Luis_Programmatic with a cognitive service Luis with Tier price S0 using di subscription key?

Thanks

Change to make enum more expressive and consistent with API documentation

The method IntentService.GetAllIntentsAsync is missing the controls to allow pagination.

?skip={skip}&take={take}

Hi Andre,

First of all, thanks for awesome library.
I just noted one small bug in the ServiceClient class.

throw new Exception($"{exception.Error.Code} - {exception.Error?.Message ?? exception.Message}");

You verify if "Error" is null before trying to get the value of the Error.Message property, but you don't do the same for Error.Code. In this case, the application will throw a NullReferenceException instead of the expected one.

Cheers.

I'm trying to use GetAllEntitiesAsync (luisId, luisVersion) but it does not work in my case:
IReadOnlyCollection entities = await luisClient.GetAllEntitiesAsync(_config.LuisId, _config.LuisVersion);
returns always a collection with 0 items... Why is that? I've checked and for instance with GetAllIntentsAsync the _config.LuisId and _config.LuisVersion work.

Check the best way to return the final status of a training request by changing an existing resource or by creating a new resource

Hi,
Is there a way to force the training in the moment I call the TrainAsync(...) method without queuing it? Or to wait until the training is finished successfully? And the same thing with publishing the model?
I would like to be sure that when I start these activities at a certain time they will be completed.
Thanks

Hi,

I have a problem when trying to access the appID of my LUIS app.
I double checked the subscription key and the location is set to WestEurope.

What can be the problem for the error below? Do i have to do any additional configuration before creating an LuisProgClient() object?