Skip to content

Jupyter Sandbox and other updates#459

Merged
jairovelasquez merged 2 commits intomasterfrom
changelog-and-jupyter
Mar 4, 2026
Merged

Jupyter Sandbox and other updates#459
jairovelasquez merged 2 commits intomasterfrom
changelog-and-jupyter

Conversation

@jairovelasquez
Copy link
Collaborator

@jairovelasquez jairovelasquez commented Mar 4, 2026

Summary by CodeRabbit

  • Documentation

    • Expanded sandbox documentation with multiple runtime types (Jupyter, EC2, AWS Console) and configuration details
    • Updated supported languages for code freezing
    • Removed outdated AWS Cloud Learning setup documentation
    • Removed file upload limit warning
  • New Features

    • Added Jupyter Sandbox support
    • Added VM image creation from Sandbox Preview
    • Added ability to create Sandboxes from templates
  • Bug Fixes

    • Corrected language file name in supported languages list

@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 75a487de-5ab6-4282-a36e-4d31c900c2f8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

This PR updates instructor documentation across multiple areas: expands Jupyter sandbox configuration details, corrects language extension references, adds new changelog entries for 2026 features, removes legacy AWS Cloud Learning documentation, and eliminates an outdated upload limit warning.

Changes

Cohort / File(s) Summary
Jupyter Sandbox Expansion
source/instructors/admin/sandboxes.rst
Significantly expanded documentation to cover Jupyter sandbox runtime type alongside existing AWS options. Added detailed configuration sections for Jupyter parameters (instance_type, custom_image, volume_size), priming options, and a cooldown note between launches. Includes new "Create Sandbox" button usage snippets with imagery.
Language Support Corrections
source/instructors/authoring/guides/freeze_code.rst
Corrected supported language file extensions: removed ocaml entry, fixed misspelled "coffescript" to proper .coffee extension, and reordered entries for consistency.
Changelog Updates
source/instructors/getstarted/support/changelog.rst
Added new March 2026 entries (Jupyter Sandboxes feature, VM image creation improvement) and February 2026 entries (Sandbox creation from templates, connectivity detection). Fixed typo in February 2018 header ("Febuary" → "February") and added missing regrade fix item.
Documentation Cleanup
source/instructors/setupcourses/awscloudlearning.rst, source/instructors/setupcourses/common-storage.rst
Removed legacy AWS Cloud Learning setup documentation and eliminated outdated 5 GB file upload limit warning from common storage documentation.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • Sandboxes V1 #436: Modifies source/instructors/admin/sandboxes.rst documentation—directly overlaps with Jupyter sandbox feature expansion in this PR.
  • Storage and more fixes #427: Modifies source/instructors/setupcourses/common-storage.rst—directly overlaps with the upload limit warning removal in this PR.

Suggested reviewers

  • shajason
  • LolaValente
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Jupyter Sandbox and other updates' is vague and generic, using the phrase 'and other updates' which lacks specificity about the actual changes in the PR. Revise the title to be more specific and descriptive. For example: 'Add Jupyter Sandbox runtime support and update documentation' or 'Document Jupyter Sandboxes and remove deprecated AWS Cloud Learning setup.'
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jairovelasquez
Copy link
Collaborator Author

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 5

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@source/instructors/admin/sandboxes.rst`:
- Line 106: Replace the inline code-style label `Create Sandbox` with the same
bold UI-label formatting used elsewhere (e.g., change `Create Sandbox` to
**Create Sandbox**) so it matches the surrounding UI action styling like **New
Collection**; update the text containing "Create Sandbox" accordingly in the
instructors/admin/sandboxes.rst content.
- Around line 65-68: Update the note text in the sandbox documentation block to
read: "There is a 15-minute cooldown period between sandbox launches." Replace
the current sentence that uses "in between Sandbox launches" with this wording
to fix casing and tighten phrasing in the .. note:: section.
- Around line 167-170: The "Required" column entry for the cloud_account field
is inconsistent: change the lowercase "optional" to a capitalized "Optional" in
the cloud_account documentation entry so it matches the other entries; locate
the cloud_account table row (the block showing ``cloud_account`` and its
attributes) and update the "Required" value to "Optional".

In `@source/instructors/authoring/guides/freeze_code.rst`:
- Line 31: The single-line extension block in
source/instructors/authoring/guides/freeze_code.rst (the long space-separated
list of extensions like .c .cpp .java .py ...) is hard to scan; break it into a
multi-line format by replacing that single-line literal with either a bullet
list (one extension per line or grouped by language family) or a fenced/indented
code literal block showing one extension per line, keeping the exact extensions
intact and preserving reStructuredText formatting so readers can quickly find
entries.

In `@source/instructors/getstarted/support/changelog.rst`:
- Line 24: The February 2026 changelog entry `[I] [24th]`: "Improved the way we
we detect an internet connection." contains a duplicated word; edit the entry
(the line starting with `[I] [24th]`) to remove the extra "we" so it reads
"Improved the way we detect an internet connection." and save the updated
source/instructors/getstarted/support/changelog.rst.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 27ed1230-114f-4a13-9c20-e17b01bf1518

📥 Commits

Reviewing files that changed from the base of the PR and between 8835fe8 and 5374e60.

⛔ Files ignored due to path filters (1)
  • source/img/sandbox_create.png is excluded by !**/*.png, !**/*.png
📒 Files selected for processing (5)
  • source/instructors/admin/sandboxes.rst
  • source/instructors/authoring/guides/freeze_code.rst
  • source/instructors/getstarted/support/changelog.rst
  • source/instructors/setupcourses/awscloudlearning.rst
  • source/instructors/setupcourses/common-storage.rst
💤 Files with no reviewable changes (2)
  • source/instructors/setupcourses/awscloudlearning.rst
  • source/instructors/setupcourses/common-storage.rst

Copy link
Collaborator

@LolaValente LolaValente left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just have one comment with the note box directly under the important box at the start of the page. Would recommend moving to the bottom of the "Key Concepts" table. Not a big deal and is ok either way :)

@jairovelasquez jairovelasquez merged commit 1659145 into master Mar 4, 2026
1 check passed
@jairovelasquez jairovelasquez deleted the changelog-and-jupyter branch March 4, 2026 18:41
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