Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 3, 2025

The HTMLRenderer now accepts a resample_interval parameter to handle large sessions (>100k samples), but the Profiler convenience methods weren't exposing it, causing test failures.

Changes

  • Added resample_interval parameter to Profiler.output_html(), Profiler.write_html(), and Profiler.open_in_browser()
  • Parameters are passed through to the HTMLRenderer constructor

Usage

profiler = Profiler()
# ... profile code ...

# Control resampling behavior for large sessions
profiler.output_html(resample_interval=0.01)  # Use specific interval
profiler.write_html('output.html', resample_interval=0)  # Disable resampling
profiler.open_in_browser(resample_interval=None)  # Auto mode (default)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Make requested changes to #407 Add resample_interval parameter to Profiler HTML output methods Nov 3, 2025
Copilot AI requested a review from joerick November 3, 2025 09:36
@joerick joerick marked this pull request as ready for review November 3, 2025 10:12
@joerick joerick merged commit 06ff905 into limit-html-sample-count Nov 3, 2025
1 check 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