Skip to content

Conversation

mawad-amd
Copy link
Collaborator

Motivation

Check deployment at https://mawad-amd.github.io/iris/

Technical Details

There are some duplicate documentation at the moment but I will leave that for later.

Test Plan

Test Result

Submission Checklist

@Copilot Copilot AI review requested due to automatic review settings September 2, 2025 11:38
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a comprehensive documentation website for the Iris multi-GPU programming framework. The website provides structured documentation including getting started guides, conceptual explanations, API references, and examples.

Key changes include:

  • Addition of Sphinx-based documentation infrastructure with automated builds
  • Comprehensive API documentation with improved docstrings
  • Website structure with installation guides, programming model explanations, and examples

Reviewed Changes

Copilot reviewed 23 out of 33 changed files in this pull request and generated no comments.

Show a summary per file
File Description
iris/iris.py Enhanced with comprehensive module and class docstrings for API documentation
iris/init.py Added module-level documentation with framework overview
docs/ Complete Sphinx documentation site with configuration, content, and build scripts
.readthedocs.yaml ReadTheDocs configuration for external documentation hosting
.github/workflows/docs.yml GitHub Actions workflow for automated documentation building and deployment

@mawad-amd
Copy link
Collaborator Author

@astroC86 I can't request your review but please check out the PR and feel free to leave comments. We would appreciate your input.

@astroC86
Copy link
Contributor

astroC86 commented Sep 2, 2025

looks very well done and polished!

@neoblizz
Copy link
Member

neoblizz commented Sep 2, 2025

This is awesome 🎉

@mawad-amd mawad-amd merged commit 8c933f0 into ROCm:main Sep 4, 2025
6 of 7 checks passed
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.

4 participants