Update README.md - #6
Conversation
I fixed the example execution command because it was missing the `config/` directory path, which caused an obvious error.
There was a problem hiding this comment.
🟡 Changes recommended
The updated commands still reference a config filename that does not exist in feature-decoding/config/, so copy/paste usage will continue to fail.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates the feature-decoding documentation to make the example commands runnable by correcting the relative path to the config YAMLs.
Changes:
- Prefixes example config file arguments with
config/in the training and prediction commands.
File summaries
| File | Description |
|---|---|
| feature-decoding/README.md | Fixes the example command arguments to reference configs via the config/ directory. |
Review details
Suppressed comments (1)
feature-decoding/README.md:55
- This example references a config file name that does not exist in
feature-decoding/config/(it appears to be missing the-imagesegments used by the actual config YAMLs). Updating to an existing config filename will prevent copy/paste failures.
python scripts/predict_feature_fastl2lir.py config/train-3d-natural-objects_rep3_fmap_test-3d-natural-objects_rep8_fmap_fmriprep_5000voxel_atlasnet.yaml
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🔵 Needs a closer look
The updated decode example points to a config filename that does not exist under feature-decoding/config/, so the copy/paste command will still fail.
Review details
Suppressed comments (1)
feature-decoding/README.md:56
- The decode example references a config file name that doesn't exist in
feature-decoding/config/(it omits-imagein both the training and test dataset segments). This will cause a file-not-found error when users copy/paste the command.
```bash
python scripts/predict_feature_fastl2lir.py config/train-3d-natural-objects_rep3_fmap_test-3d-natural-objects_rep8_fmap_fmriprep_5000voxel_atlasnet.yaml
- **Files reviewed:** 1/1 changed files
- **Comments generated:** 0 new
- **Review effort level:** Lite
</details>
I fixed the example execution command because it was missing the
config/directory path, which caused an obvious error.