Skip to content

Profile zstash check#450

Draft
forsyth2 wants to merge 21 commits into
mainfrom
issue446-profile-check
Draft

Profile zstash check#450
forsyth2 wants to merge 21 commits into
mainfrom
issue446-profile-check

Conversation

@forsyth2

@forsyth2 forsyth2 commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Objectives:

  • Add performance profiling for zstash check.

Issue resolution:

Select one: This pull request is...

  • a bug fix: increment the patch version
  • a small improvement: increment the minor version
  • a new feature: increment the minor version
  • an incompatible (non-backwards compatible) API change: increment the major version

Small Change

  • To merge, I will use "Squash and merge". That is, this change should be a single commit.
  • Logic: I have visually inspected the entire pull request myself.
  • Pre-commit checks: All the pre-commits checks have passed.

@forsyth2 forsyth2 self-assigned this Jun 11, 2026
@forsyth2 forsyth2 added the Testing Files in `tests` modified label Jun 11, 2026
@forsyth2

Copy link
Copy Markdown
Collaborator Author

The first 17 commits are from #427. They will be removed from this PR once that PR is merged and I rebase this branch.

@forsyth2 forsyth2 changed the title Issue446 profile check Profile zstash check Jun 11, 2026
@forsyth2

forsyth2 commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator Author

Prototype of Fig 3 (check):

image

Remaining action items:

@forsyth2

Copy link
Copy Markdown
Collaborator Author

Prototype of Fig 3b (the comparison) using yesterday's main-branch/Unified baseline and a "current" of the #447's code (the zstash check speedup PR):

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Testing Files in `tests` modified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Improve efficiency of zstash check

1 participant