Skip to content

Fix typos in documentation - #422

Merged
cregouby merged 5 commits into
mlverse:mainfrom
raus7n:docs/fix-typos
Oct 26, 2026
Merged

cregouby merged 5 commits into
mlverse:mainfrom
raus7n:docs/fix-typos

Conversation

@raus7n

@raus7n raus7n commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

I fixed a few spelling mistakes in the documentation, code comments, and vignette.

Changes made:

  • assesment → assessment in R/collection-rf100-damage.R
  • intializer → initializer in R/models-alexnet.R
  • internaly → internally in R/models-fcn.R
  • bouding → bounding in R/ops-box_convert.R
  • occuring → occurring in a code comment in R/transforms-tensor.R
  • handeling → handling (3 occurrences) in vignettes/torchvision-classes.Rmd

I found these with codespell, checked each one manually, regenerated the affected help pages (rf100_damage_collection, model_alexnet, model_fcn_resnet, box_xywh_to_xyxy) using devtools::document() with roxygen2 8.1.1, and added an entry to NEWS.md.

One thing I left unchanged:

codespell also flags classses in the 0.9.0 section of NEWS.md. I left it as it is since it's part of an older release, but I'm happy to fix it if needed.

These are just spelling and documentation fixes. No code logic or functionality was changed.

@cregouby

Copy link
Copy Markdown
Collaborator

Hello @raus7n,
This is awesome, thanks !
Please proceed for typo in the News as well.
I'll add a commit to align the FR translation.

@raus7n

raus7n commented Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

@cregouby Thanks for the feedback and for updating the FR translation! I’ve fixed the classses typo in NEWS.md, and also changed (@318) to (#318) on the same line since it refers to PR #318.

@cregouby
cregouby merged commit 8d09e25 into mlverse:main Oct 26, 2026
@raus7n
raus7n deleted the docs/fix-typos branch October 26, 2026 19:14
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.

2 participants