Skip to content

Update citation nature genetics#8

Merged
Arvindiyer merged 2 commits into
mainfrom
update-citation-nature-genetics
Jul 11, 2026
Merged

Update citation nature genetics#8
Arvindiyer merged 2 commits into
mainfrom
update-citation-nature-genetics

Conversation

@Arvindiyer

@Arvindiyer Arvindiyer commented Jul 11, 2026

Copy link
Copy Markdown
Member

Summary

Adds the citation for the published Nature Genetics paper to the SelectSim repository.

Changes

  • Updated README.md with the Nature Genetics paper citation.
  • Added the recommended citation information for users of SelectSim.
  • Removed tracked compiled object files from src/ and added appropriate ignore rules where needed.

Test plan

  • devtools::document() run to regenerate NAMESPACE and manual pages.
  • devtools::check() completed successfully.
  • No package functionality was changed.
  • No new tests were required because this update is documentation-only.
  • README documentation updated with the published paper citation.

Additional notes

This pull request does not change package behavior or analysis results. It only updates the repository documentation to reference the published Nature Genetics article.

Summary by CodeRabbit

  • Documentation
    • Updated citation information across the README and package citation metadata.
    • Added the finalized Nature Genetics publication details, including authors, year, and DOI.
    • Recorded the citation update in the release notes for version 0.1.6.

Fill in journal, DOI, and full author list in inst/CITATION and the
README citation blocks, replacing the TBD placeholders now that the
manuscript is published as Iyer et al. (2026), Nature Genetics,
doi:10.1038/s41588-026-02661-4.
@coderabbitai

coderabbitai Bot commented Jul 11, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The change finalizes the Nature Genetics citation across package metadata, README files, and release notes, and updates .gitignore patterns for compiled artifacts under src/.

Changes

Publication citation updates

Layer / File(s) Summary
Finalize package citation metadata
inst/CITATION
Updates the author list, journal, year, DOI, DOI URL, and formatted citation text.
Synchronize citation documentation
README.Rmd, README.md, NEWS.md
Replaces placeholder citation details with the finalized Nature Genetics publication and DOI.

Source artifact ignore rules

Layer / File(s) Summary
Update compiled artifact patterns
.gitignore
Adds patterns for compiled object and library artifacts and an accompanying unignore rule.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: updating citations to the Nature Genetics paper.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-citation-nature-genetics

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.gitignore:
- Line 15: Correct the object-file ignore rule in .gitignore: replace the
negated nonexistent .githubsrc pattern with an ignore pattern for object files
under src/, using src/*.o or src/**/*.o if nested build outputs are possible.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e3ac76fd-ddfd-4b25-abac-ed1a09d43d98

📥 Commits

Reviewing files that changed from the base of the PR and between 94fcabf and e139627.

⛔ Files ignored due to path filters (2)
  • src/RcppExports.o is excluded by !**/*.o
  • src/code.o is excluded by !**/*.o
📒 Files selected for processing (5)
  • .gitignore
  • NEWS.md
  • README.Rmd
  • README.md
  • inst/CITATION

Comment thread .gitignore
.*
!/.gitignore
!/.github No newline at end of file
!/.githubsrc/*.o

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Fix the object-file ignore pattern.

!/.githubsrc/*.o targets a nonexistent .githubsrc path and negates the rule instead of ignoring object files under src/. Consequently, src/*.o artifacts can still be committed.

-!/.githubsrc/*.o
+src/*.o

If nested build outputs are possible, use src/**/*.o instead.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
!/.githubsrc/*.o
src/*.o
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.gitignore at line 15, Correct the object-file ignore rule in .gitignore:
replace the negated nonexistent .githubsrc pattern with an ignore pattern for
object files under src/, using src/*.o or src/**/*.o if nested build outputs are
possible.

@Arvindiyer Arvindiyer merged commit 9e11f9e into main Jul 11, 2026
5 checks passed
@Arvindiyer Arvindiyer deleted the update-citation-nature-genetics branch July 11, 2026 01:02
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.

1 participant