Repository navigation
APPHY 275 (Fall 2026) — Code Server sub-app - #17
Merged
Merged
Conversation
This was referenced Sep 23, 2026
kgray-hu
marked this pull request as ready for review
September 30, 2026 15:03
kgray-hu
requested review from
Tanvez,
arthurian and
jaguillette
and
a balanced review from Copilot
September 30, 2026 15:11
Contributor
Author
|
@jaguillette @Tanvez @arthurian - For APPHY275 (CodeServer), this has been tested in prod, wiill merge this into get this to the faculty for testing. But have tagged you all to take a look when you get a chance. |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The configuration parses correctly and matches the stated access and resource requirements.
Review effort: Balanced
Findings: None
What changed in this PR
Adds the APPHY 275 Code Server sub-app for Fall 2026.
Changes:
- Restricts visibility to Canvas course
170782. - Provides general-queue sessions with 1–4 CPUs and 1–6 hours.
- Disables session caching.
| File | Description |
|---|---|
local/apphy275.yml.erb |
Adds the course-specific configuration and access controls. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Overview
Re-adds the generic Code Server course entry for APPHY 275, which was removed during the Spring
2026 cleanup. It has no course-specific software customization; the entry records use by this
course and gives students a convenient terminal on a
generalnode for submitting batch work.Changes
local/apphy275.yml.erb, recovered from the Spring 2026 release tag and repointed fromCanvas
165547to170782.general, permits 1–4 CPUs, defaults to twohours (six maximum), and starts a fresh session each time.
diff -u local/generic.yml.erb local/apphy275.yml.erbdiff -u spring-2026:local/apphy275.yml.erb local/apphy275.yml.erbNotes
course folder and course Spack. It was the tested path for submitting and observing QE jobs.
loading the course QE installation, run
spack env deactivate; unset SPACK_ENV; the QE testscript does this.
images, or environments.
Testing
general-dy-general-cr-1and used its terminal.rc=0; course files were readable but notstudent-writable, and QE loaded. The instructor's job 825 confirmed that the three restricted
queues were denied.
Course-member access test
Script:
course-folder/tests/access/run-access-test.shResult:
===== ACCESS TEST DONE rc=0 =====. The script checked membership, shared home,student ability to read, but not change, course assets; both environment copies; the QE Spack load;
allowed queues (
general,gpu,desktop); and denied restricted queues. Staff are expected todiffer only where the course folder is deliberately staff-writable.
general) and 803 (gpu); both passedall seven checks and reported
-15.80731203 Ry.Quantum ESPRESSO test
Script:
course-folder/tests/qe/run-qe-test.shResult:
===== QE TEST DONE rc=0 =====. The log confirmed the requested MPI ranks,pseudopotential, SCF convergence, total energy, and
JOB DONE. The script leaves the app's Spacksetup, loads the QE build that matches the current CPU type, and uses
mpirunwith one processingthread for each CPU requested by the job.
References
#9,#14,#16Harvard-ATG/ood-misc-runbooks#61