progress: indicatif bar in eigenvalue loop + per-case/seed banners - #11
Merged
Merged
Conversation
Wires indicatif into both CPU and GPU eigenvalue power-iteration
loops so long simulations no longer run silent. The new
`transport::progress::EigenProgress` exposes three sinks:
- animated bar (TTY) — `[████░░░░] 47/150 | k=1.00123
<k>=1.00098 +/- 64 pcm | ETA 2m13s`
- per-batch eprintln (forced) — opt-in via OPEN_RUST_MC_PROGRESS=1
when stderr is piped (cargo test
--nocapture, CI, run_benchmark.ps1
tee'd output)
- silent — default off-TTY behaviour, keeps
sweep logs lean
Auto-hides under verbose=true so it doesn't fight with println! debug
output.
Per-seed banners added to the ICSBEP integration tests
(tests/icsbep_runs.rs::run_case_e2e_seeds and
tests/cuda_runs.rs::run_case_cuda_seeds) print
`[icsbep] case -- seed N/M starting` + elapsed-time done line around
each seed; useful when single seeds take ≥400 s.
Python sweep (icsbep_sweep.py) gains a per-case banner
`[idx/total] case starting (Nseed x Bb x Ii x Pp)` and per-seed
banners inside run_case_multi_seed. run_benchmark.ps1 gains a
-LiveProgress switch that sets OPEN_RUST_MC_PROGRESS=1 so per-batch
lines land in the .log (since Tee-Object masks the TTY).
Tests:
- cargo test --lib --release: 438 / 438
- cargo test --lib --release --features cuda: 451 / 451
- tests/icsbep_runs::heu_met_fast_001_case_1_godiva smoke (PASS)
- python icsbep_sweep.py smoke on heu-met-fast-001 (PASS)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Wires indicatif into both CPU and GPU eigenvalue power-iteration loops so long simulations no longer run silent. The new
transport::progress::EigenProgressexposes three sinks:[████░░░░] 47/150 | k=1.00123 <k>=1.00098 +/- 64 pcm | ETA 2m13swhen stderr is piped (cargo test
--nocapture, CI, run_benchmark.ps1
tee'd output)
sweep logs lean
Auto-hides under verbose=true so it doesn't fight with println! debug output.
Per-seed banners added to the ICSBEP integration tests (tests/icsbep_runs.rs::run_case_e2e_seeds and
tests/cuda_runs.rs::run_case_cuda_seeds) print
[icsbep] case -- seed N/M starting+ elapsed-time done line around each seed; useful when single seeds take ≥400 s.Python sweep (icsbep_sweep.py) gains a per-case banner
[idx/total] case starting (Nseed x Bb x Ii x Pp)and per-seed banners inside run_case_multi_seed. run_benchmark.ps1 gains a -LiveProgress switch that sets OPEN_RUST_MC_PROGRESS=1 so per-batch lines land in the .log (since Tee-Object masks the TTY).Tests: