Enhanced TuiMonitor display to include Current Testcase Index and use… #2810
Conversation
|
Thanks! Can you |
| if let Some(stat) = client.get_user_stats("Current Testcase Index") { | ||
| write!(fmt, ", Testcase Index: {}", stat.value()).unwrap(); | ||
| } else { | ||
| write!(fmt, ", Testcase Index: N/A").unwrap(); |
There was a problem hiding this comment.
No need to show anything if it's not available.
Also, this only ends up in the log, not in the TUI, right?
There was a problem hiding this comment.
Yes, this code update ensures that the "Testcase Index" information only ends up in the log, not in the TUI. ok will remove else
There was a problem hiding this comment.
I think user stats already end up in the log (not 100% sure though)
There was a problem hiding this comment.
(and really, it'd be nice to have this in the UI for the individual fuzzer, that's the idea of the tui, but we can do that in another PR)
There was a problem hiding this comment.
i can try in this PR to add this in the UI for the individual fuzzer
| let corpus_idx_value = corpus_idx.0; // Extract `usize` value from `CorpusId` | ||
|
|
||
| // Fire the UpdateUserStats event with the corpus index | ||
| _manager.fire( |
There was a problem hiding this comment.
This should be optional (i.e., something you set in the builder), it may lead to a lot of events so slow down the fuzzer
There was a problem hiding this comment.
ok thanks will look into this
|
Any news? :) |
Fixes : #2757 (comment)
Description :
This PR enhances the
TuiMonitorto display the "Current Testcase Index" and user-defined stats in thedisplaymethod. These updates improve usability by providing more detailed real-time insights into the monitoring process.