Skip to content

Add progress bar for multiple iterations#20

Closed
delalamo wants to merge 3 commits intomainfrom
claude/add-tqdm-progress-bar-t5NnA
Closed

Add progress bar for multiple iterations#20
delalamo wants to merge 3 commits intomainfrom
claude/add-tqdm-progress-bar-t5NnA

Conversation

@delalamo
Copy link
Copy Markdown
Owner

Show progress bars when iteration counts are greater than one:

  • Pipeline outputs loop (n_outputs > 1)
  • Pipeline iterations loop (n_iterations > 1)
  • AMBER relaxation outer iterations (max_outer_iterations > 1)
  • AMBER minimization retry attempts (max_attempts > 1)
  • Side-chain denoising steps (num_denoising_steps > 1)

claude and others added 3 commits January 18, 2026 19:18
Show progress bars when iteration counts are greater than one:
- Pipeline outputs loop (n_outputs > 1)
- Pipeline iterations loop (n_iterations > 1)
- AMBER relaxation outer iterations (max_outer_iterations > 1)
- AMBER minimization retry attempts (max_attempts > 1)
- Side-chain denoising steps (num_denoising_steps > 1)
Replace multiple nested progress bars with a single unified bar that
tracks overall progress across all outputs and iterations. The bar
shows:
- Total progress: output X/Y, iter A/B
- Current phase: designing/repacking/relaxing

Changes:
- pipeline.py: Create single progress bar with total = n_outputs * n_iterations
- designer.py: Accept pbar parameter, pass to pack_side_chains
- relaxer.py: Accept pbar parameter, pass to AmberRelaxation
- relax.py: Accept pbar parameter, pass to amber_minimize
- amber_minimize.py: Accept pbar parameter, remove standalone progress bars
- sc_utils.py: Accept pbar parameter, remove standalone progress bar
delalamo pushed a commit that referenced this pull request Jan 23, 2026
The progress bar feature added in PR #20 requires tqdm, but it was not
added to the dependencies list. This caused the integration tests to
fail with an ImportError when importing graphrelax modules.

https://claude.ai/code/session_01QUT5Lv9GyV2Qs35A4TbxZg
@delalamo
Copy link
Copy Markdown
Owner Author

Refer to PR #28

@delalamo delalamo closed this Jan 23, 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