Closed giotab closed 1 month ago
@giotab , thanks for your contribution! I added a few comments ππΌ
@giotab , thanks for your contribution! I added a few comments ππΌ
Hi @martedesco , not sure I can see the comments π€ unless you mean in the PR's description?
Also, should I bump the version number to 1.5.0
?
Hi @martedesco , not sure I can see the comments π€ unless you mean in the PR's description?
My bad, I didn't submit them!
Also, should I bump the version number to 1.5.0?
I will add a few things before releasing it, so no need to worry about that ππΌ
In order to see Editors usage as well, I changed the existing LanguagesBreakdown component to be more generic so that both Languages and Editors require less effort to maintain.
Here are the most significant changes:
New Features:
src/components/BreakdownsComponent.vue
: Introduced a new component that provides a generic breakdown view. This component can display breakdowns by different keys such as language or editor. It includes a pie chart to visualize the top 5 items by accepted prompts and acceptance rate, and a data table for a detailed breakdown.Enhancements:
src/components/MainComponent.vue
: Replaced theLanguagesBreakdown
component with the newBreakdownsComponent
. Added 'editors' to the list of tab items to provide a breakdown by editors. [1] [2] [3]src/components/MetricsViewer.vue
: Removed the processing of language breakdowns as it is now handled by theBreakdownsComponent
. [1] [2]Refactoring:
src/model/Breakdown.ts
: Renamed fromLanguage.ts
and updated the class name toBreakdown
to make it more generic.src/model/Metrics.ts
: Renamed theBreakdown
class toBreakdownData
and updated thebreakdown
member of theMetrics
class to useBreakdownData
. [1] [2] [3]Removals:
src/components/LanguagesBreakdown.vue
: Removed this component as its functionality is now covered by theBreakdownsComponent
.src/components/MetricsViewer.vue
: Removed the import of theLanguage
model as it is no longer used.