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.

Sign up for GitHub

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

General: Display total LLM cost as text instead of pie chart #10088

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
+7 −17
Open

General: Display total LLM cost as text instead of pie chart #10088

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
10d92bd
Display total LLM cost in course detail view
wasnertobias Dec 13, 2024
f217a6c
Merge branch 'develop' into feature/course-details-show-llm-usage
wasnertobias Dec 14, 2024
88dc737
Costs are in EUR, not in USD
wasnertobias Dec 14, 2024
de75832
Merge branch 'develop' into feature/course-details-show-llm-usage
wasnertobias Dec 16, 2024
4bc3d38
Adds index on course_id to optimize the cost calculation query
wasnertobias Dec 17, 2024
a89db19
Show LLM cost info only if any LLM feature is enabled in course.
wasnertobias Dec 17, 2024
56c8672
Show LLM cost info in text instead of pie chart.
wasnertobias Dec 30, 2024
2f87092
Merge branch 'develop' into feature/course-details-show-llm-usage
wasnertobias Dec 30, 2024
a1bbdd6
Merge branch 'develop' into feature/course-details-show-llm-usage
wasnertobias Jan 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 2 additions & 10 deletions src/main/webapp/app/course/manage/detail/course-detail-doughnut-chart.component.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,16 +10,8 @@ <h5 class="text-center">{{ 'artemisApp.course.detail.' + doughnutChartTitle + 'T
<div class="doughnut-chart-container" [ngClass]="titleLink ? 'clickable' : ''" (click)="openCorrespondingPage()">
<div class="doughnut-chart-text">
@if (receivedStats) {
@if (currentPercentage != undefined) {
<div class="h4">{{ currentPercentage }}%</div>
} @else if (showText) {
<div class="h4">{{ showText }}</div>
}
@if (!showText) {
<div class="h6">{{ currentAbsolute }} / {{ currentMax }}</div>
} @else if (currentPercentage != undefined) {
<div class="h6">{{ showText }}</div>
}
<div class="h4">{{ currentPercentage }}%</div>
<div class="h6">{{ currentAbsolute }} / {{ currentMax }}</div>
} @else {
<div class="h3">
<fa-icon [icon]="faSpinner" animation="spin" />
Expand Down
12 changes: 5 additions & 7 deletions src/main/webapp/app/course/manage/detail/course-detail.component.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,13 +41,11 @@
[currentMax]="courseDTO?.currentMaxAverageScore"
/>
@if (this.irisChatEnabled || this.isAthenaEnabled) {
<jhi-course-detail-doughnut-chart
class="doughnut-container col-sm-6 my-3"
[course]="course"
[contentType]="DoughnutChartType.CURRENT_LLM_COST"
[showText]="courseDTO?.currentTotalLlmCostInEur + ' €'"
[currentMax]="0"
/>
<div class="doughnut-container col-sm-6 my-3">
<h5 class="text-center">
{{ 'artemisApp.course.detail.currentTotalLLMCostTitle' | artemisTranslate }}{{ ': ' + courseDTO?.currentTotalLlmCostInEur + ' €' }}
</h5>
</div>
}
</div>
</div>
Expand Down
Loading