Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: NVIDIA/cuPhoton/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: NVIDIA/cuPhoton/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (68)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 WalkthroughWalkthroughThe pull request standardizes component-name capitalization across documentation, examples, source text, generated labels, and tests. It also adds detail to xPois input and xDR setup descriptions, updates xScan feature-bundle documentation, and deletes ROADMAP.md. ChangesComponent name standardization
Roadmap removal
Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to No actionable merge-blocking risk was identified. 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
Comment |
tpn
marked this pull request as ready for review
September 24, 2026 03:16
tpn
changed the title
tpn
changed the title
tpn
force-pushed
the
release-readme-updates
branch
from
September 24, 2026 22:54
ec942a5 to
d473b69
Compare
Collaborator
Author
|
@coderabbitai review |
✅ Action performedReview finished.
|
tpn
force-pushed
the
release-readme-updates
branch
2 times, most recently
from
September 28, 2026 20:55
4ce31fd to
8ca3452
Compare
tpn
changed the base branch from
main
to
codex/imaging-fit-visualizations-20260928
September 28, 2026 20:55
tpn
force-pushed
the
release-readme-updates
branch
from
September 28, 2026 21:22
8ca3452 to
234b934
Compare
tpn
changed the base branch from
codex/imaging-fit-visualizations-20260928
to
main
September 28, 2026 21:22
tpn
force-pushed
the
release-readme-updates
branch
from
September 28, 2026 22:11
234b934 to
9e6a5e7
Compare
Keep the introduction focused on the reference workflows and their integration points before the workflow chooser. Signed-off-by: Trent Nelson <trentn@nvidia.com>
Retire the v0.1.0 roadmap as part of the release documentation cleanup. Track planned work in GitHub issues instead of maintaining a separate list of release intentions. Signed-off-by: Trent Nelson <trentn@nvidia.com>
Spell component names with a lowercase x across guides, examples, CLI help, reports, and diagnostic messages. Update matching test expectations while preserving identifiers and artifact contracts. Signed-off-by: Trent Nelson <trentn@nvidia.com>
Align the guides with FITS reader policies, candidate inputs, device execution, native packaging, and xRay analysis contracts. Add release notes and correct installation, citation, and command examples. Signed-off-by: Trent Nelson <trentn@nvidia.com>
tpn
changed the title
tpn
force-pushed
the
release-readme-updates
branch
from
September 29, 2026 03:38
9e6a5e7 to
c82949e
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The guides had fallen behind the upcoming 0.1.3 release: shared FITS reader selection, direct FITS candidate workflows, persistent GPU execution, and native packaging were missing or described inconsistently. Update the README, changelog, component guides, CLI help, installation instructions, and artifact contracts against the current implementation. Clarify reader fallback, device/host result boundaries, xRay fit granularity and units, and what the documented validation commands actually exercise.
Preserve this PR's component-name cleanup and removal of the README maturity disclaimer and outdated roadmap, including the removal requested in #58. The reviewed imaging walkthrough from #67 is preserved. Python changes affect documentation, comments, and user-facing names; identifiers, algorithms, artifact schemas, and notebook executable content are unchanged. Replaces #52.
Validation:
make ci-lintincludes the all-files pre-commit run;make package-checkandgit diff --checkpass.GPU, native-GDS, and multi-node execution were not rerun for this documentation pass. Release installation instructions remain conditional on publication.