From 5132996c77c101f816a91e5a5e6287db6052e0ca Mon Sep 17 00:00:00 2001
From: Landon <100798954+landonmsft@users.noreply.github.com>
Date: Fri, 14 Feb 2025 13:11:37 -0500
Subject: [PATCH] Pass Feature Name as Telemetry for NL2Fx (#2848)
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.
---
.../Protocol/CustomNL2FxParams.cs | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/src/libraries/Microsoft.PowerFx.LanguageServerProtocol/Protocol/CustomNL2FxParams.cs b/src/libraries/Microsoft.PowerFx.LanguageServerProtocol/Protocol/CustomNL2FxParams.cs
index bb9ee7be67..2f7bc7923e 100644
--- a/src/libraries/Microsoft.PowerFx.LanguageServerProtocol/Protocol/CustomNL2FxParams.cs
+++ b/src/libraries/Microsoft.PowerFx.LanguageServerProtocol/Protocol/CustomNL2FxParams.cs
@@ -21,6 +21,11 @@ public class CustomNL2FxParams : BaseNLParams, IHasTextDocument
/// Sentence in Natural Language.
///
public string Sentence { get; set; }
+
+ ///
+ /// Name of the NL2Fx feature being used.
+ ///
+ public string FeatureName { get; set; }
}
///