Skip to content

Benchmark wrt input size#224

Merged
jtec merged 9 commits into
mainfrom
benchmark_wrt_input_size
May 22, 2026
Merged

Benchmark wrt input size#224
jtec merged 9 commits into
mainfrom
benchmark_wrt_input_size

Conversation

@jtec
Copy link
Copy Markdown
Owner

@jtec jtec commented May 21, 2026

Makes the PRX bechmark a bit more interesting by

  • Profiling execution time across a range of input obs file durations
  • Profiling peak RAM usage
  • Plotting execution time per function, making it easy to see how individual functions scale (we like O(n) - and O(1) even more ;) and where prx spends its time (spoiler alert: parsing the nav file)

Example outcome with a 3h, 1Hz input file

Screenshot 2026-05-22 at 22 20 38

@jtec jtec marked this pull request as ready for review May 21, 2026 15:56
@jtec jtec requested a review from plutonheaven May 21, 2026 15:57
- fix path detection on Windows
- fix new prx_src_directory call
@plutonheaven
Copy link
Copy Markdown
Collaborator

memray does not work on Windows, so I added some fixes to ignore it on Windows. The benchmark will be less complete...

I also corrected a call to prx_root_directory which has just been changed to prx_src_directory and also made a small change to OS-independent path.

@jtec
Copy link
Copy Markdown
Owner Author

jtec commented May 22, 2026

@plutonheaven Thanks for testing on Windows and making the fix!

@jtec jtec merged commit 7416062 into main May 22, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants