docs(P4): update README.md with S3DIS experiments and remove UTF-8 characters - #10
Conversation
|
Important Review skipped You've hit the daily PR analysis limit on your current plan. Consider upgrading: Upgrade ⚙️ SettingsSeverity Threshold: 📖 User Guide
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughREADME.md was reformatted with neutral headings, expanded documentation navigation, reorganized visual results, and added detailed S3DIS benchmark, ablation, and label-efficiency results. ChangesREADME Documentation Refresh
Estimated code review effort: 2 (Simple) | ~10 minutes ✨ 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 |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the README.md file by removing emojis from headers, updating formatting, and adding a comprehensive section on S3DIS Area-5 evaluation results, including model benchmarks, an RGB ablation study, and a label efficiency study. The review feedback suggests improving usability by converting inline image paths to clickable markdown links and fixing a minor capitalization inconsistency ("Labeled" to "labeled").
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| | hybrid | 0.2749 | 0.6582 | | ||
| | hybrid_v2 | 0.2986 | 0.6636 | | ||
|
|
||
| Detailed per-class IoU results are documented in [docs/EXPERIMENTS.md](docs/EXPERIMENTS.md). A visualization of the per-class IoU bar chart can be found at `docs/images/per_class_iou.png`. |
There was a problem hiding this comment.
Consider converting the inline code path to a clickable markdown link so that readers can easily navigate to the visualization on GitHub.
| Detailed per-class IoU results are documented in [docs/EXPERIMENTS.md](docs/EXPERIMENTS.md). A visualization of the per-class IoU bar chart can be found at `docs/images/per_class_iou.png`. | |
| Detailed per-class IoU results are documented in [docs/EXPERIMENTS.md](docs/EXPERIMENTS.md). A visualization of the per-class IoU bar chart can be found at [docs/images/per_class_iou.png](docs/images/per_class_iou.png). |
| - **20 rooms**: mIoU 0.2561 | ||
| - **40 rooms**: mIoU 0.2558 | ||
|
|
||
| The learning curve plot is located at `docs/images/label_efficiency.png`. |
There was a problem hiding this comment.
Consider converting the inline code path to a clickable markdown link so that readers can easily navigate to the learning curve plot on GitHub.
| The learning curve plot is located at `docs/images/label_efficiency.png`. | |
| The learning curve plot is located at [docs/images/label_efficiency.png](docs/images/label_efficiency.png). |
| Every run produces a standard artifacts bundle in the `outputs/` directory: | ||
|
|
||
| - **`segmented_room.ply`**: Fully labeled 3D point cloud. | ||
| - **`segmented_room.ply`**: Fully Labeled 3D point cloud. |
There was a problem hiding this comment.
For consistency with the other bullet points in this section, the word 'Labeled' should be lowercase ('labeled').
1 participant
Updates README.md to remove all UTF-8/emoji characters (ensuring 100% ASCII) and documents the Phase 4 S3DIS evaluation/experiment results.
Summary by CodeRabbit