mirror of
https://github.com/aljazceru/Auto-GPT.git
synced 2025-12-17 22:14:28 +01:00
Implement UI Disable Feature During Benchmark Run
Added a state variable isBenchmarkRunning in SkillTreeViewModel to track the status of benchmark execution. This state variable is used to conditionally disable specific UI components: - The "Initiate test suite" button in TaskQueueView is disabled during the benchmark. - All IconButtons in SideBarView are disabled during the benchmark. - Node selection in SkillTreeView is disabled during the benchmark. This ensures that the user cannot interact with these components while a benchmark test is running, thereby improving UX and preventing potential issues.
This commit is contained in:
@@ -9,11 +9,15 @@ import 'package:flutter/services.dart';
|
||||
import 'package:graphview/GraphView.dart';
|
||||
|
||||
class SkillTreeViewModel extends ChangeNotifier {
|
||||
// TODO: Potentially move to task queue view model when we create one
|
||||
final BenchmarkService benchmarkService;
|
||||
// TODO: Potentially move to task queue view model when we create one
|
||||
bool isBenchmarkRunning = false;
|
||||
|
||||
List<SkillTreeNode> _skillTreeNodes = [];
|
||||
List<SkillTreeEdge> _skillTreeEdges = [];
|
||||
SkillTreeNode? _selectedNode;
|
||||
// TODO: Potentially move to task queue view model when we create one
|
||||
List<SkillTreeNode>? _selectedNodeHierarchy;
|
||||
|
||||
SkillTreeNode? get selectedNode => _selectedNode;
|
||||
@@ -69,6 +73,7 @@ class SkillTreeViewModel extends ChangeNotifier {
|
||||
}
|
||||
|
||||
void toggleNodeSelection(String nodeId) {
|
||||
if (isBenchmarkRunning) return;
|
||||
if (_selectedNode?.id == nodeId) {
|
||||
// Unselect the node if it's already selected
|
||||
_selectedNode = null;
|
||||
@@ -127,6 +132,9 @@ class SkillTreeViewModel extends ChangeNotifier {
|
||||
|
||||
// TODO: Update to actual implementation
|
||||
Future<void> runBenchmark(ReportRequestBody reportRequestBody) async {
|
||||
isBenchmarkRunning = true;
|
||||
notifyListeners();
|
||||
|
||||
try {
|
||||
final result = await benchmarkService.generateReport(reportRequestBody);
|
||||
// Pretty-print the JSON result
|
||||
@@ -135,6 +143,9 @@ class SkillTreeViewModel extends ChangeNotifier {
|
||||
} catch (e) {
|
||||
print("Failed to generate report: $e");
|
||||
}
|
||||
|
||||
isBenchmarkRunning = false;
|
||||
notifyListeners();
|
||||
}
|
||||
|
||||
// TODO: Update to actual implementation
|
||||
|
||||
@@ -1,4 +1,6 @@
|
||||
import 'package:auto_gpt_flutter_client/viewmodels/skill_tree_viewmodel.dart';
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:provider/provider.dart';
|
||||
|
||||
class SideBarView extends StatelessWidget {
|
||||
final ValueNotifier<String> selectedViewNotifier;
|
||||
@@ -7,6 +9,9 @@ class SideBarView extends StatelessWidget {
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
// TODO: should we pass this in as a dependency?
|
||||
final skillTreeViewModel =
|
||||
Provider.of<SkillTreeViewModel>(context, listen: false);
|
||||
return Material(
|
||||
child: ValueListenableBuilder(
|
||||
valueListenable: selectedViewNotifier,
|
||||
@@ -21,7 +26,9 @@ class SideBarView extends StatelessWidget {
|
||||
color:
|
||||
selectedView == 'TaskView' ? Colors.blue : Colors.black,
|
||||
icon: const Icon(Icons.chat),
|
||||
onPressed: () => selectedViewNotifier.value = 'TaskView',
|
||||
onPressed: skillTreeViewModel.isBenchmarkRunning
|
||||
? null
|
||||
: () => selectedViewNotifier.value = 'TaskView',
|
||||
),
|
||||
IconButton(
|
||||
splashRadius: 0.1,
|
||||
@@ -29,8 +36,9 @@ class SideBarView extends StatelessWidget {
|
||||
? Colors.blue
|
||||
: Colors.black,
|
||||
icon: const Icon(Icons.emoji_events), // trophy icon
|
||||
onPressed: () =>
|
||||
selectedViewNotifier.value = 'SkillTreeView',
|
||||
onPressed: skillTreeViewModel.isBenchmarkRunning
|
||||
? null
|
||||
: () => selectedViewNotifier.value = 'SkillTreeView',
|
||||
),
|
||||
],
|
||||
),
|
||||
|
||||
@@ -53,17 +53,19 @@ class TaskQueueView extends StatelessWidget {
|
||||
border: Border.all(color: Colors.green, width: 3),
|
||||
),
|
||||
child: ElevatedButton(
|
||||
onPressed: () {
|
||||
// Create a ReportRequestBody with hardcoded values
|
||||
ReportRequestBody reportRequestBody = ReportRequestBody(
|
||||
category: "",
|
||||
tests: testNames,
|
||||
mock: true,
|
||||
);
|
||||
onPressed: viewModel.isBenchmarkRunning
|
||||
? null
|
||||
: () {
|
||||
// Create a ReportRequestBody with hardcoded values
|
||||
ReportRequestBody reportRequestBody = ReportRequestBody(
|
||||
category: "",
|
||||
tests: testNames,
|
||||
mock: true,
|
||||
);
|
||||
|
||||
// Call runBenchmark method from SkillTreeViewModel
|
||||
viewModel.runBenchmark(reportRequestBody);
|
||||
},
|
||||
// Call runBenchmark method from SkillTreeViewModel
|
||||
viewModel.runBenchmark(reportRequestBody);
|
||||
},
|
||||
child: Row(
|
||||
mainAxisAlignment:
|
||||
MainAxisAlignment.center, // Center the children
|
||||
|
||||
Reference in New Issue
Block a user