Port missing LuisTelemetryConstants for 'topTwoIntents' from C#

Is your feature request related to a problem? Please describe. Currently, we are only able to log the top scoring intent to AppInsights via the integrated TelemetryClient in the LuisRecognizer. In C#, there is the option to pass TelemetryProperties that enable the logging of both the top scoring intent and the second highest scoring intent. This is not yet implemented in botbuilder-js.

Describe the solution you’d like Port the LuisTelemetryConstants Intent2Property and IntentScore2Property from the C# luis telemetry constants. To enable usage of those constants, the corresponding properties will also have to be defined in LuisRecognizer.fillTelemetryProperties(). This can also be ported from the C# project.

Also, the LuisTelemetryConstants are currently not exported from the package. This would have to be mitigated as well in order to enable easy use of them.

Describe alternatives you’ve considered The current workaround is to create a custom telemetry event and log those properties there. However, this is additional code and effort that would not be necessary if those LuisTelemetryConstants were implemented.

[enhancement]

[jwiley84]

Hi @mrestorff Just wanted to let you know this is still on my plate. We’re discussing it internally, and you should see a PR with the changes in a few days. Thank you for your patience in this.

[joshgummersall]

@jwiley84 do you have any updates on this issue?