Skip to content

Fix Title typo, add contributors, and correct stale package links - #28

Merged
jananiravi merged 1 commit into
mainfrom
readme_desc_fixes
Jul 24, 2026
Merged

Fix Title typo, add contributors, and correct stale package links#28
jananiravi merged 1 commit into
mainfrom
readme_desc_fixes

Conversation

@eboyer221

Copy link
Copy Markdown
Contributor

Summary

Metadata and README cleanup for amRml ahead of the bioRxiv preprint and Bioconductor submission. Documentation only — no functional code changes.

Changes

  • DESCRIPTION: fix typo in Title — "Machile learning" -> "Machine learning"
  • DESCRIPTION: add Emily Boyer, Charmie Vang, and Alexander McKim (ORCID 0000-0002-7802-7591) as ctb
  • DESCRIPTION: amRshiny -> amRviz in the Description field
  • README: fix the amRdata link (it pointed at amRml) and amRshiny -> amRviz
  • README: replace the plain-text citation block with the formatted bioRxiv citation (DOI 10.64898/2026.07.10.734579), marking Ghosh & Brenner as co-first authors

Review notes

  • Please confirm the author list, name spellings, and ORCID are correct.
  • README.md is regenerated from README.Rmd; both are included.

Fix typo in Package Title (Machile → Machine). Add contributors Emily Boyer, Charmie Vang, and Alexander McKim (with ORCID) to Authors@R. Replace amRshiny references with amRviz and correct the amRdata GitHub link in README files. Update the citation text to the bioRxiv entry with DOI and add a co-first authors note.

@amcim amcim left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Straightforward text edits to the DESCRIPTION and readme. Approving changes

CI fails, but that is because of the vip issue. This will be a separate PR

@jananiravi
jananiravi merged commit 39c78f9 into main Jul 24, 2026
0 of 7 checks passed
@jananiravi
jananiravi deleted the readme_desc_fixes branch July 24, 2026 20:45
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.

3 participants