Skip to content

Enable IR parallel by default in CPU Apptainer image - #106

Merged
jonathanschultzNU merged 2 commits into
mainfrom
cursor/freq-parallel-cpu-sif-12c1
Aug 30, 2026
Merged

Enable IR parallel by default in CPU Apptainer image#106
jonathanschultzNU merged 2 commits into
mainfrom
cursor/freq-parallel-cpu-sif-12c1

Conversation

@jonathanschultzNU

Copy link
Copy Markdown
Collaborator

Summary

The parallel IR-intensity CPU worker path already shipped in #102. This PR turns it on by default for the classroom CPU image and aligns the Settings UI with deployment policy.

  • quantui.def sets QUANTUI_FREQ_PARALLEL=1 in %environment
  • Settings tab: checkbox shows the effective value and is disabled while the env var is set, with a short deployment note
  • Site-wide off: set QUANTUI_FREQ_PARALLEL=0 in the image or OnDemand env (overrides saved settings, same as before)
  • GPU SIF unchanged — parallel IR stays opt-in there

Students on NCShare need no launcher changes; the OnDemand session inherits the variable automatically.

Testing

pytest tests/test_freq_ir_workers.py tests/test_cpu_container_def.py -q -n0

34 passed.

Open in Web Open in Cursor 

NCCU-Schultz-Lab and others added 2 commits August 30, 2026 20:58
The parallel IR-intensity worker path shipped in #102; this turns it on
for classroom CPU images via QUANTUI_FREQ_PARALLEL=1 in quantui.def.
Settings reflects the deployment value and locks the checkbox while the
env var is set (use QUANTUI_FREQ_PARALLEL=0 to force off site-wide).

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
end-of-file-fixer on tests/test_freq_ir_workers.py; ruff import order
in app.py.

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
@jonathanschultzNU
jonathanschultzNU marked this pull request as ready for review August 30, 2026 21:57
@jonathanschultzNU
jonathanschultzNU merged commit 6ee2968 into main Aug 30, 2026
5 checks passed
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