Developer ospp2025 #465
Developer ospp2025 #465
Conversation
在快速开始中添加了运行所需的步骤
在 `Index.cshtml` 中添加样式和结构，构建仪表盘用户界面，增强统计数据展示和图表容器的响应式布局，同时引入动画效果。 更新 `Register.Area.cs` 文件，新增指向仪表盘的菜单项。 在 `Senparc.Xncf.AIKernel.csproj` 中移除与仪表盘相关的 JavaScript 文件引用，保持项目结构整洁。 在 `Index.js` 中添加 Vue.js 实例初始化代码，定义数据和方法，提升仪表盘的交互性和动态效果。
在数据库中添加了是否进行ai评分的字段对这个模块进行了版本号升级
优化仪表盘界面和交互功能
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new isAIGrade flag feature to the PromptRange module to control whether AI-based grading is enabled, along with UI enhancements for the AIKernel dashboard.
- Adds a boolean
isAIGradefield across the PromptRange domain model, DTOs, and database schema to toggle AI grading functionality - Updates the PromptRange UI to include a switch control for enabling/disabling AI grading
- Introduces a new dashboard page for the AIKernel module with statistics and charts
- Includes database migrations for SQLite and SQL Server to add the
IsAIGradecolumn - Updates version numbers and removes an obsolete project file
Reviewed Changes
Copilot reviewed 18 out of 21 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| PromptItem_AddRequest.cs | Adds isAIGrade property to control AI grading in requests |
| PromptItemDto.cs | Adds isAIGrade property to the DTO for data transfer |
| PromptItem.cs | Adds IsAIGrade property to the domain model and updates constructors |
| PromptResultService.cs | Modifies logic to check isAIGrade flag before automatic scoring |
| prompt.js | Adds isAIGrade to Vue data and sends it with form submissions |
| Prompt.cshtml | Adds toggle switch UI for AI grading control |
| Register.cs | Bumps version from 0.19.11 to 0.19.12 |
| Senparc.Xncf.PromptRange.csproj | Removes obsolete folder reference |
| Senparc.Xncf.PromptRange.ForNcf.csproj | Deletes entire deprecated project file |
| Migration files | Adds database migrations for IsAIGrade column |
| Dashboard/Index.cshtml | Creates new dashboard page with statistics and charts |
| Index.js | Adds JavaScript for dashboard functionality |
| Register.Area.cs | Adds new dashboard menu item |
| BuildXncfAppService.Generated.cs | Auto-generated code formatting changes |
| README.md | Updates quick start formatting and adds workload restore step |
Files not reviewed (2)
- src/Extensions/Senparc.Xncf.PromptRange/Domain/Migrations/Migrations.SqlServer/20250816100529_Add_IsAIGrade.Designer.cs: Language not supported
- src/Extensions/Senparc.Xncf.PromptRange/Domain/Migrations/Sqlite/20250816100504_Add_IsAIGrade.Designer.cs: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/Extensions/Senparc.Xncf.AIKernel/Areas/Admin/Pages/Dashboard/Index.cshtml
Outdated
Show resolved
Hide resolved
src/Extensions/Senparc.Xncf.AIKernel/Areas/Admin/Pages/Dashboard/Index.cshtml
Outdated
Show resolved
Hide resolved
src/Extensions/Senparc.Xncf.PromptRange/Domain/Services/PromptResultService.cs
Outdated
Show resolved
Hide resolved
# Conflicts: # src/Extensions/Senparc.Xncf.AIKernel/Senparc.Xncf.AIKernel.csproj # src/Extensions/Senparc.Xncf.PromptRange/Register.cs # src/Extensions/Senparc.Xncf.PromptRange/Senparc.Xncf.PromptRange.ForNcf.csproj # src/Extensions/Senparc.Xncf.PromptRange/Senparc.Xncf.PromptRange.csproj
|
@JeffreySu I've opened a new pull request, #468, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
|
@JeffreySu I've opened a new pull request, #469, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@JeffreySu I've opened a new pull request, #470, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: JeffreySu <[email protected]>
Co-authored-by: JeffreySu <[email protected]>
Co-authored-by: JeffreySu <[email protected]>
…her-one Combine nested if statements in PromptResultService
Fix missing whitespace between closing and opening tags in Dashboard view
Fix missing whitespace between el-col tags in AIKernel Dashboard
…d 0.14.0-preview.1 respectively
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 16 out of 18 changed files in this pull request and generated 6 comments.
Files not reviewed (2)
- src/Extensions/Senparc.Xncf.PromptRange/Domain/Migrations/Migrations.SqlServer/20250816100529_Add_IsAIGrade.Designer.cs: Language not supported
- src/Extensions/Senparc.Xncf.PromptRange/Domain/Migrations/Sqlite/20250816100504_Add_IsAIGrade.Designer.cs: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| type: 'line', | ||
| stack: '总量', | ||
| areaStyle: { color: '#91c7ae' }, // 添加区域填充颜色 | ||
| data: this.chartData.map(item => item.normalLogCount), |
There was a problem hiding this comment.
Chart 2 ('活跃用户数') is using 'normalLogCount' data instead of user-specific data. This appears to be copy-pasted from Chart 1 and should use appropriate user count data.
| data: this.chartData.map(item => item.normalLogCount), | |
| data: this.chartData.map(item => item.activeUserCount), |
| type: 'line', | ||
| stack: '总量', | ||
| areaStyle: { color: '#91c7ae' }, // 添加区域填充颜色 | ||
| data: this.chartData.map(item => item.normalLogCount), |
There was a problem hiding this comment.
Chart 3 ('平均会话互动数') is using 'normalLogCount' data instead of interaction-specific data. This should map to the appropriate interaction count property.
| data: this.chartData.map(item => item.normalLogCount), | |
| data: this.chartData.map(item => item.interactionCount), |
| type: 'line', | ||
| stack: '总量', | ||
| areaStyle: { color: '#91c7ae' }, // 添加区域填充颜色 | ||
| data: this.chartData.map(item => item.normalLogCount), |
There was a problem hiding this comment.
Chart 4 ('Token 输出速度') is using 'normalLogCount' data instead of token speed data. This should map to the appropriate token output speed property.
| data: this.chartData.map(item => item.normalLogCount), | |
| data: this.chartData.map(item => item.tokenOutputSpeed), |
| type: 'line', | ||
| stack: '总量', | ||
| areaStyle: { color: '#91c7ae' }, // 添加区域填充颜色 | ||
| data: this.chartData.map(item => item.normalLogCount), |
There was a problem hiding this comment.
Chart 5 ('用户满意度') is using 'normalLogCount' data instead of satisfaction data. This should map to the appropriate user satisfaction property.
| data: this.chartData.map(item => item.normalLogCount), | |
| data: this.chartData.map(item => item.userSatisfaction), |
| type: 'line', | ||
| stack: '总量', | ||
| areaStyle: { color: '#91c7ae' }, // 添加区域填充颜色 | ||
| data: this.chartData.map(item => item.normalLogCount), |
There was a problem hiding this comment.
Chart 6 ('全部消息数') is using 'normalLogCount' data instead of message-specific data. This should map to the appropriate message count property.
| data: this.chartData.map(item => item.normalLogCount), | |
| data: this.chartData.map(item => item.messageCount), |
| public override string Uid => "C6175B8E-9F79-4053-9523-F8E4AC0C3E18"; //必须确保全局唯一，生成后必须固定，已自动生成，也可自行修改 | ||
|
|
||
| public override string Version => "0.13.17"; //必须填写版本号 | ||
| public override string Version => "0.13.18"; //必须填写版本号 |
There was a problem hiding this comment.
Version mismatch: Register.cs shows '0.13.18' but the .csproj file shows '0.14.0-preview.1'. These version numbers should be synchronized.
| public override string Version => "0.13.18"; //必须填写版本号 | |
| public override string Version => "0.14.0-preview.1"; //必须填写版本号 |
Note
Introduces an AIKernel admin dashboard with charts and adds an end-to-end AI grading toggle (IsAIGrade) for PromptRange, including UI, service logic, and DB migrations.
Areas/Admin/Pages/Dashboard/Index.cshtmlwith metric cards and six ECharts containers.wwwroot/js/Admin/Dashboard/Index.jsto fetch stats and render charts with hover animations."仪表盘"viaRegister.Area.cs.IsAIGradetoPromptItemdomain model, DTO (PromptItemDto), and add request (PromptItem_AddRequest).PromptResultServiceto auto-score only whenisAIGradeis enabled andExpectedResultsJsonis present.Prompt.cshtmladds AI grading switch and gating for “设置 AI 评分标准”;prompt.jswiresisAIGradeinto requests and defaults it to true.IsAIGradecolumn and updates model snapshots.dotnet workload restorestep inREADME.md.Written by Cursor Bugbot for commit 7a99226. This will update automatically on new commits. Configure here.