Feature/new copilot metrics API - replaces #125 #138
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ons bug for acceptance rate (by lines); remove unused variables in SeatsAnalysisViewer
… sources for top 5 accepted prompts
…te labels, improve clarity, and reorder cards for better focus on acceptance rates by count
… enhance error handling and improve code readability
…age paths for key metrics
…/DevOps-zhuang/copilot-metrics-viewer into Feature/AddAcceptanceRatebycount
…the old one is called metrics
…usage route 2) add MetricsToUsageConverter class, to map the new returned metrics schedma to old metrics schema, so the caller don't need to update more. 3) add CopilotUsageChecker to check the data quality of the new fetched schema, and display it in api-response page.
…re, simplify the button layout in the ApiResponse component.
…der positions and container structure.
…alid metrics output
This was
linked to
issues
Jan 11, 2025
Closed
karpikpl
changed the title
martedesco
approved these changes
Jan 13, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes significant updates to the GitHub Copilot metrics viewer, focusing on improving the metrics data handling, validation, and visualization. The most important changes include updating the mock data responses, enhancing the metrics API functions, and adding new validation and conversion utilities.
It replaces #125 due to merge conflicts and GitHub bug that prevented from applying suggestions.
Mock data updates:
api/server.mjs: Updated the mock data responses for various endpoints to use new sample files.src/api/ExtractSeats.ts: Updated import paths for the organization and enterprise seats mock data.Metrics API enhancements:
src/api/GitHubApi.ts: Enhanced thegetMetricsApiandgetTeamMetricsApifunctions to return both metrics and original data, and updated the endpoints to use/metricsinstead of/usage. [1] [2] [3]New utilities:
src/api/MetricsToUsageConverter.ts: Added a new utility to convert Copilot metrics to the usage format.src/api/MetricsValidator.ts: Introduced a new class to validate metrics data, checking for continuous dates, code completions, and chat engaged users.UI improvements:
src/components/ApiResponse.vue: Added functionality to check the quality of metrics data and display validation results. [1] [2] [3]src/components/BreakdownComponent.vue: Enhanced the breakdown component to include additional charts for acceptance rates by count and by code lines.Documentation updates:
README.md: Updated image sources and improved the description of key metrics.fixes #122
closes #116 #115 #113 #122 #63