Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Telemetry] Add button click event tracking for main menu and error analysis component #1570

Merged
merged 13 commits into from
Jul 21, 2022

Conversation

tongyu-microsoft
Copy link
Contributor

This PR adds button click and combo box selection event log for main menu and error analysis component. We will use telemetryHook to pass event names to AzureML studio UI, so that we can keep tracking of what buttons users click on.

Description

Here is list of events that we will be tracking:
// Error analysis
ErrorAnalysisTreeMapTabClick = "ErrorAnalysisTreeMapTabClick",
ErrorAnalysisTreeMapFeatureListClick = "ErrorAnalysisTreeMapFeatureListClick",
ErrorAnalysisTreeMapClearSelection = "ErrorAnalysisTreeMapClearSelection",
ErrorAnalysisTreeMapSaveAsNewCohortClick = "ErrorAnalysisTreeMapSaveAsNewCohortClick",
ErrorAnalysisTreeMapCohortSaved = "ErrorAnalysisTreeMapCohortSaved",
ErrorAnalysisHeatMapTabClick = "ErrorAnalysisHeatMapTabClick",
ErrorAnalysisHeatMapQuantileBinningClick = "ErrorAnalysisHeatMapQuantileBinningClick",
ErrorAnalysisNewMetricSelected = "ErrorAnalysisNewMetricSelected",
// Main menu
MainMenuCohortSettingsClick = "MainMenuCohortSettingsClick",
MainMenuDashboardConfigurationClick = "MainMenuDashboardConfigurationClick",
MainMenuSwitchCohortClick = "MainMenuSwitchCohortClick",
MainMenuNewCohortClick = "MainMenuNewCohortClick",
// Cohort
NewCohortAdded = "NewCohortAdded"

Checklist

  • I have added screenshots above for all UI changes.
  • I have added e2e tests for all UI changes.
  • Documentation was updated if it was needed.

@codecov-commenter
Copy link

codecov-commenter commented Jul 20, 2022

Codecov Report

Merging #1570 (c396062) into main (39b285e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1570   +/-   ##
=======================================
  Coverage   88.03%   88.03%           
=======================================
  Files         109      109           
  Lines        5291     5291           
=======================================
  Hits         4658     4658           
  Misses        633      633           
Flag Coverage Δ
unittests 88.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39b285e...c396062. Read the comment docs.

@vinuthakaranth
Copy link
Contributor

Thanks for adding these trackers! Really helpful for debugging customer issues!!

@tongyu-microsoft tongyu-microsoft enabled auto-merge (squash) July 21, 2022 18:44
1 similar comment
@tongyu-microsoft tongyu-microsoft merged commit 0d5bda5 into main Jul 21, 2022
@tongyu-microsoft tongyu-microsoft deleted the tongy/addButtonClickLog branch July 21, 2022 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants