ENH: Factor tutorial dataset access into helper #43
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
docs/tutorial/models.md
Outdated
| from nifreeze.data.splitting import lovo_split | ||
| from nireports.reportlets.modality.dwi import plot_dwi | ||
| from tutorial_data import load_tutorial_dmri_dataset | ||
| dmri_dataset = DWI.from_filename("../../data/dwi.h5") | ||
| dmri_dataset = load_tutorial_dmri_dataset() |
There was a problem hiding this comment.
The notebooks now rely on from tutorial_data import load_tutorial_dmri_dataset, but tutorial_data.py lives in the repository root and is neither installed nor added to PYTHONPATH. When the pages are executed from docs/tutorial (the default working directory for jupyter-book and for users running a single notebook), only that directory is on sys.path, so the import raises ModuleNotFoundError and the build fails before any cells run. Consider moving the helper into the docs tree or explicitly adding the repo root to sys.path before the import.
Useful? React with 👍 / 👎.
oesteban
changed the title
Summary
dwi_full_brainmask.h5tutorial_datahelper that caches the DWI dataset and exposes loading utilitiesTesting
https://chatgpt.com/codex/tasks/task_e_68e59037bbc88330aec3ed01b3bceb52