Skip to content

Commit

Permalink
Pass Feature Name as Telemetry for NL2Fx (#2848)
Browse files Browse the repository at this point in the history
Currently for the telemetry object in NL2Fx, we only pass one unified
identifier for all NL2Fx features which results in data not being
classified at the feature level. We must change that, for better
telemetry information.

This is a four-part change. In these repo, Power-Fx (GitHub), I need to
write a variable in CustomNL2Fx, to pass to PPUX.
  • Loading branch information
landonmsft authored Feb 14, 2025
1 parent de2fbd0 commit 5132996
Showing 1 changed file with 5 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,11 @@ public class CustomNL2FxParams : BaseNLParams, IHasTextDocument
/// Sentence in Natural Language.
/// </summary>
public string Sentence { get; set; }

/// <summary>
/// Name of the NL2Fx feature being used.
/// </summary>
public string FeatureName { get; set; }
}

/// <summary>
Expand Down

0 comments on commit 5132996

Please sign in to comment.