docs: consolidate the getting-started cluster — quickstart, learning ladder, archive - #1165
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
Warning Review limit reached
Next review available in: 6 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe documentation is reorganized around current tutorials, archived workshop snapshots, and a new model-comparison how-to. Navigation and redirects are updated. Notebook checks exclude archived notebooks and the computationally heavy comparison notebook. ChangesDocumentation consolidation
Estimated code review effort: 4 (Complex) | ~45 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (3)
docs/how_to/compare_models.ipynb (2)
83-92: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick winFix the
matplotlibalias and the conflicting backend calls.Line 83 binds the top-level
matplotlibpackage to the nameplt. The conventional meaning ofpltismatplotlib.pyplot. The cell runs becausematplotlib.useexists, but any reader who copies this setup and then callsplt.subplots()gets anAttributeError.Line 91 selects the
Aggbackend, and line 92 then switches to the inline backend. TheAggcall has no effect.♻️ Proposed setup cell
# Import modules import arviz as az import jax -import matplotlib as plt import pytensor import hssm pytensor.config.floatX = "float32" jax.config.update("jax_enable_x64", False) -plt.use("Agg") %matplotlib inline %config InlineBackend.figure_format='retina' - -# hssm.set_floatX("float32")🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/how_to/compare_models.ipynb` around lines 83 - 92, Update the notebook setup imports to bind pyplot as plt, and remove the conflicting matplotlib.use("Agg") call so the existing %matplotlib inline directive is the sole backend selection. Preserve the remaining imports and configuration statements.
397-419: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winRemove the commented-out simulation code.
Lines 397-419 hold about 23 lines of disabled code from the workshop notebook. The active list comprehension at lines 422-428 already builds the dataset. This is a new curated how-to, not an archived snapshot, so the dead code adds noise for readers.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/how_to/compare_models.ipynb` around lines 397 - 419, Remove the commented-out simulate_participant3 examples for dataset2_participant1 through dataset2_participant16, including the surrounding severity and diagnosis headings. Keep the active dataset-building list comprehension unchanged.mkdocs.yml (1)
103-105: 🗄️ Data Integrity & Integration | 🔵 Trivial | ⚡ Quick winUse internal targets for the notebook redirects.
Set each target to its corresponding
archive/*.ipynbsource path. Keep the oldtutorials/*.mdkeys becausemkdocs-redirectsrequires markdown-shaped redirect keys;.ipynbkeys trigger a warning. Internal targets keep previews and fork builds self-contained.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@mkdocs.yml` around lines 103 - 105, Update the three notebook redirect values for tutorials/hssm_tutorial_workshop_1.md, tutorials/hssm_tutorial_workshop_2.md, and tutorials/pymc_to_hssm.md to their corresponding internal archive/*.ipynb source paths. Preserve the existing tutorials/*.md keys because mkdocs-redirects requires markdown-shaped redirect keys.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/archive/pymc_to_hssm.ipynb`:
- Line 43: Update both download URLs in cell 5 of the pymc_to_hssm notebook from
the removed docs/tutorials/pymc_to_hssm/ path to docs/archive/pymc_to_hssm/,
preserving the rest of the links and notebook content.
In `@docs/how_to/compare_models.ipynb`:
- Around line 312-317: Add deterministic random seeding to simulate_participant3
before its np.random.normal calls, ensuring repeated notebook executions
generate the same dataset, model fits, and ELPD values referenced in the
narrative.
---
Nitpick comments:
In `@docs/how_to/compare_models.ipynb`:
- Around line 83-92: Update the notebook setup imports to bind pyplot as plt,
and remove the conflicting matplotlib.use("Agg") call so the existing
%matplotlib inline directive is the sole backend selection. Preserve the
remaining imports and configuration statements.
- Around line 397-419: Remove the commented-out simulate_participant3 examples
for dataset2_participant1 through dataset2_participant16, including the
surrounding severity and diagnosis headings. Keep the active dataset-building
list comprehension unchanged.
In `@mkdocs.yml`:
- Around line 103-105: Update the three notebook redirect values for
tutorials/hssm_tutorial_workshop_1.md, tutorials/hssm_tutorial_workshop_2.md,
and tutorials/pymc_to_hssm.md to their corresponding internal archive/*.ipynb
source paths. Preserve the existing tutorials/*.md keys because mkdocs-redirects
requires markdown-shaped redirect keys.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: d36c3646-29cb-4128-9a34-33d52003c605
⛔ Files ignored due to path filters (31)
docs/archive/hssm_tutorial_workshop_1/ssm_overview.pngis excluded by!**/*.pngdocs/archive/hssm_tutorial_workshop_2/Choice_Learning.csvis excluded by!**/*.csvdocs/archive/hssm_tutorial_workshop_2/HDDM_results.pngis excluded by!**/*.pngdocs/archive/hssm_tutorial_workshop_2/Palmer_etal_2024.PNGis excluded by!**/*.pngdocs/archive/hssm_tutorial_workshop_2/behavioral_task.pngis excluded by!**/*.pngdocs/archive/hssm_tutorial_workshop_2/choiceRatios.pngis excluded by!**/*.pngdocs/archive/hssm_tutorial_workshop_2/ddm_schematic.pngis excluded by!**/*.pngdocs/archive/hssm_tutorial_workshop_2/latencies.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/example_network/lan/ddm/a227477e683411f0abcd6ae25f443f62_lan_ddm_network_config.pickleis excluded by!**/*.pickledocs/archive/pymc_to_hssm/example_network/lan/ddm/ddm_lan_a227477e683411f0abcd6ae25f443f62_data_details.pickleis excluded by!**/*.pickledocs/archive/pymc_to_hssm/example_network/lan/ddm/ddm_lan_a227477e683411f0abcd6ae25f443f62_train_config.pickleis excluded by!**/*.pickledocs/archive/pymc_to_hssm/example_network/lan/ddm/ddm_lan_a227477e683411f0abcd6ae25f443f62_training_history.csvis excluded by!**/*.csvdocs/archive/pymc_to_hssm/images/ANGLE_with_params_pic.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/DDM_with_params_pic.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/Experiment.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/HSSM_logo.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/HSSM_logo_old.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/Race_3.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/basic_lan_pic.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/network_hssm_pymc.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/network_onnx.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/onnx_logo.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/playground.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/pymc_logo.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/race_with_difficulty.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/rv_to_hssm.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/simulation_to_network.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/images/zeus_sampler.pngis excluded by!**/*.pngdocs/archive/pymc_to_hssm/mathpsych_workshop_2025_data/mathpsych_2025_full.parquetis excluded by!**/*.parquetdocs/archive/pymc_to_hssm/mathpsych_workshop_2025_data/mathpsych_2025_modeling.parquetis excluded by!**/*.parquetdocs/archive/pymc_to_hssm/mathpsych_workshop_2025_data/mathpsych_2025_parameters.pklis excluded by!**/*.pkl
📒 Files selected for processing (19)
.github/workflows/check_notebooks.ymldocs/archive/hssm_tutorial_workshop_1.ipynbdocs/archive/hssm_tutorial_workshop_2.ipynbdocs/archive/pymc_to_hssm.ipynbdocs/archive/pymc_to_hssm/example_network/lan/ddm/ddm_lan_a227477e683411f0abcd6ae25f443f62_model.onnxdocs/archive/pymc_to_hssm/example_network/lan/ddm/ddm_lan_a227477e683411f0abcd6ae25f443f62_train_state_dict.ptdocs/archive/pymc_to_hssm/mathpsych_workshop_2025_data/ddm_lan.onnxdocs/archive/pymc_to_hssm/mathpsych_workshop_2025_data/ddm_lan_batch.onnxdocs/archive/pymc_to_hssm/mathpsych_workshop_2025_data/race_3_no_bias_lan.onnxdocs/archive/pymc_to_hssm/mathpsych_workshop_2025_data/race_3_no_bias_lan_batch.onnxdocs/archive/pymc_to_hssm/network_config.yamldocs/getting_started/getting_started.ipynbdocs/how_to/compare_models.ipynbdocs/how_to/specify_priors.ipynbdocs/index.mddocs/tutorials/ddm_hierarchical_tutorial.ipynbdocs/tutorials/main_tutorial.ipynbdocs/tutorials/scientific_workflow_hssm.ipynbmkdocs.yml
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The flagship PR of the docs-overhaul Phase 2: six pages re-teaching the same simulate → define → sample → PPC workflow become a four-rung learning ladder plus an archive. No unique content is lost — everything unique was extracted first.
getting_started.ipynbslimmed from 41 cells / 4 model fits to an 18-cell, single-fit 15-minute quickstart, re-executed fresh, ending in a posterior predictive check (hssm.plotting.plot_predictive) and an explicit next-steps ladder. 2.1 MB → 0.7 MB.how_to/specify_priors.ipynb) — the prior-syntax walkthrough extracted from the old getting-started (include dicts,hssm.Param,hssm.Prior, bounds, fixing, shortcut syntax, regression-coefficient priors), re-executed fresh.how_to/compare_models.ipynb) — seeded from workshop 1'saz.comparepassage (a top demand gap); adds the interpretation guidance the workshop never had (elpd_diff vs. dse, weights, Pareto-k caveats). Stored outputs inherited from the workshop run; sampler progress spam trimmed; skipped in notebook CI like its source was.docs/archive/with event-snapshot banners crediting the original authors; redirects cover the old URLs;docs/archive/is excluded from notebook CI and docs-build execution.mkdocs build --strictgreen; quickstart + priors how-to re-executed end-to-end with zero errors.Closes #1164
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation
Tests