Skip to content
This repository was archived by the owner on Mar 5, 2026. It is now read-only.

Handle missing FPS in parser - #388

Merged
qqrm merged 1 commit into
mainfrom
codex/update-fps-handling-in-parse_perf_log.py
Aug 22, 2025
Merged

Handle missing FPS in parser#388
qqrm merged 1 commit into
mainfrom
codex/update-fps-handling-in-parse_perf_log.py

Conversation

@qqrm

@qqrm qqrm commented Aug 22, 2025

Copy link
Copy Markdown
Owner

Summary

  • write zero FPS when log lacks measurements
  • accept zero FPS in test for missing data

Testing

  • cargo fmt --all
  • cargo check --tests --benches
  • cargo clippy --tests --benches --fix --allow-dirty -- -D warnings
  • cargo test (fails: Exec format error)
  • pytest tests/parse_perf_log_script.py

https://chatgpt.com/codex/tasks/task_e_68a889720e2c8332b9b4a57a4de9774d

@qqrm
qqrm merged commit d42b320 into main Aug 22, 2025
2 of 3 checks passed
@qqrm
qqrm deleted the codex/update-fps-handling-in-parse_perf_log.py branch August 22, 2025 16:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant