Skip to content

Workflow: queue/cache memory estimates and high-memory confirmation - #186

Open
pecomyint wants to merge 2 commits into
change/workflow-add-missing-defaults-reportfrom
add/workflow-memory-estimates
Open

pecomyint wants to merge 2 commits into
change/workflow-add-missing-defaults-reportfrom
add/workflow-memory-estimates

Conversation

@pecomyint

Copy link
Copy Markdown
Collaborator

Stacked on #185 — review/merge that one first; this diff shows only this PR's change.

Summary

  • Each server-queue / collector-cache spinbox shows a 1 MP reference estimate (image and image + float64 Q).
  • Starting a consumer whose buffers would retain ≥1 GiB asks for confirmation; larger buffers do not raise sustained throughput.
  • The profile save review shows a frame-cache estimate when cache sizes change.

Test plan

  • pytest tests/unit passes; ruff check src/dashpva/ clean
  • New tests in test_workflow_memory_estimates.py

🤖 Generated with Claude Code

@pecomyint
pecomyint requested a review from Osayi-ANL September 30, 2026 22:15

@Osayi-ANL Osayi-ANL left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check out the memory-estimate readout update I added in commit c22acee. It makes the estimate compact, keeps it in muted grey italics, and bolds the calculated memory sizes. If you like the change, please re-request my review and I’ll approve.

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