Repository navigation
Fix SciTS tutorial issues found in an end-to-end Aurora run - #5
Merged
Merged
Conversation
- evaluation.md: replace the DDP/HSDP discussion with an overview of the evaluation pipeline (training signals, checkpoints, held-out scoring in the notebook) and a metrics guide; keep limitations short at the end. - README/environment: the trunk is OLMo-1B, not Qwen3-0.6B; link the notebook; add a recursive clone + submodule step (the env build fails at step 5/7 without the walrus submodule). - training.md: pass --project (defaults to ModCon). - data.md: point hackathon users at the pre-staged data and note the ~3 h full-conversion runtime. - notebook: show the effective max_steps override, hide weight-loading progress bars, install ipywidgets alongside jupyterlab. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
pemami4911
approved these changes
Oct 14, 2026
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.
Description
Summarize the change and the motivation behind it. Describe the user-visible
behavior, not just the implementation.
Fixes # (issue number)
Type of Change
Compatibility
PRISM has downstream users running long jobs on HPC systems. Does this PR
change any of the following?
src.*import pathsprismCLI surfaceIf any box above is checked, describe the migration path here.
How Has This Been Tested?
List what you ran, and what you intentionally skipped and why. Hardware-specific
results may be reported here when the required platform is unavailable to you.
New Dependencies, Data, or Assets
Any new dependency, dataset, pretrained model, tokenizer, or other asset — with
its license and provenance. Write "None" if there are none.
Checklist
bash tools/ci/local_quality.sh)(see SECURITY.md)
AI/LLM Assistance
See Guidelines for AI/LLM-Assisted Contributions in
CONTRIBUTING.md. All PRs require a human reviewer who is
accountable for the review — a project norm, not yet enforced by branch
protection.
Additional Context
Anything else a reviewer should know.