Skip to content

Add webstepper_options attribute and line_context option to SnapshotTracer#1360

Open
rachelzUT wants to merge 2 commits into
pyta-uoft:masterfrom
rachelzUT:add-webstepper-options
Open

Add webstepper_options attribute and line_context option to SnapshotTracer#1360
rachelzUT wants to merge 2 commits into
pyta-uoft:masterfrom
rachelzUT:add-webstepper-options

Conversation

@rachelzUT

@rachelzUT rachelzUT commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Proposed Changes

Added a protected attribute, webstepper_options, to SnapshotTracer. This allows users to pass configuration options for the MemoryViz Webstepper when webstepper=True, otherwise a warning is shown. So far, the only option added is the line_context option that, when greater than 0, displays that number of lines before and after the traced code block.

...

Screenshots of your changes (if applicable)

Type of Change

Type Applies?
🚨 Breaking change (fix or feature that would cause existing functionality to change)
New feature (non-breaking change that adds functionality) X
🐛 Bug fix (non-breaking change that fixes an issue)
♻️ Refactoring (internal change to codebase, without changing functionality)
🚦 Test update (change that only adds or modifies tests)
📚 Documentation update (change that only updates documentation)
📦 Dependency update (change that updates a dependency)
🔧 Internal (change that only affects developers or continuous integration)

Checklist

Before opening your pull request:

  • I have performed a self-review of my changes.
    • Check that all changed files included in this pull request are intentional changes.
    • Check that all changes are relevant to the purpose of this pull request, as described above.
  • I have added tests for my changes, if applicable.
    • This is required for all bug fixes and new features.
  • I have updated the project documentation, if applicable.
    • This is required for new features.
  • I have updated the project Changelog (this is required for all changes).
  • If this is my first contribution, I have added myself to the list of contributors.

After opening your pull request:

  • I have verified that the CI tests have passed.
  • I have reviewed the test coverage changes reported by Coveralls.
  • I have requested a review from a project maintainer.

Questions and Comments

@coveralls

coveralls commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 28262818290

Coverage remained the same at 90.931%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 4058
Covered Lines: 3690
Line Coverage: 90.93%
Coverage Strength: 17.67 hits per line

💛 - Coveralls

@rachelzUT rachelzUT requested a review from david-yz-liu June 26, 2026 20:24
@rachelzUT rachelzUT changed the title Add webstepper_options attribute and line_context option Add webstepper_options attribute and line_context option to SnapshotTracer Jul 3, 2026
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