Skip to content

Frame time from mangohud #81

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

Merged
mockersf merged 4 commits into from
Feb 21, 2026
Merged

Frame time from mangohud #81

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
cac7baa
don't keep params in graph name
mockersf Jan 18, 2026
d5ec217
display frame time from mangohud if available
mockersf Jan 18, 2026
94a0f62
alphabetical sort
mockersf Jan 18, 2026
8962688
use frame time from mangohud if available for the whole period
mockersf Feb 21, 2026
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
32 changes: 28 additions & 4 deletions src/bin/reports.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
let mut runtime = stress_tests;
runtime.extend(large_scenes);

runtime.sort_by(|a, b| a.0.cmp(&b.0));
let stress_tests_alpha = runtime
.iter()
.map(|(name, _)| name.clone())
Expand Down Expand Up @@ -235,6 +236,7 @@ fn setup_runtime(kind: &str, stats: &[Stats], cache_id: &str) -> Vec<(String, f6
timestamp: u128,
commit: String,
frame_time: u64,
frame_time_mangohud: Option<u64>,
cpu: u64,
gpu: u64,
}
Expand All @@ -255,7 +257,7 @@ fn setup_runtime(kind: &str, stats: &[Stats], cache_id: &str) -> Vec<(String, f6
let with_z_scores = stress_tests
.into_iter()
.flat_map(|stress_test| {
let values = stats
let mut values = stats
.iter()
.filter(|stat| {
(chrono::Utc::now()
Expand Down Expand Up @@ -285,6 +287,13 @@ fn setup_runtime(kind: &str, stats: &[Stats], cache_id: &str) -> Vec<(String, f6
.cloned()
.unwrap() as f64))
as u64,
frame_time_mangohud: stat
.metrics
.get(&format!(
"{kind}.{}.{}.frame_time.mean",
stress_test.0, stress_test.1
))
.cloned(),
cpu: stat
.metrics
.get(&format!(
Expand All @@ -309,10 +318,25 @@ fn setup_runtime(kind: &str, stats: &[Stats], cache_id: &str) -> Vec<(String, f6
return None;
}

// frame time from mangohud is available for all commits, use it instead
if values.iter().all(|v| v.frame_time_mangohud.is_some()) {
values.iter_mut().for_each(|v| {
v.frame_time = v.frame_time_mangohud.unwrap();
v.frame_time_mangohud = None;
});
} else {
// TODO: once this log doesn't happen, cleanup frame time computation
println!(
"Using old frame time for {} {}",
stress_test.0, stress_test.1
);
}

serde_json::to_writer(
std::fs::File::create(format!(
"data/{}_{}{cache_id}.json",
stress_test.0, stress_test.1
"data/{}{}{cache_id}.json",
stress_test.0,
stress_test.1.replace("params", ""),
))
.unwrap(),
&values,
Expand Down Expand Up @@ -349,7 +373,7 @@ fn setup_runtime(kind: &str, stats: &[Stats], cache_id: &str) -> Vec<(String, f6

with_z_scores
.into_iter()
.map(|(name, params, z_score)| (format!("{name}_{params}"), z_score))
.map(|(name, params, z_score)| (format!("{name}{}", params.replace("params", "")), z_score))
.collect()
}

Expand Down
Loading