Skip to content

Fix: item transform dataset argument - #402

Merged
cregouby merged 2 commits into
mlverse:mainfrom
srishtiii28:fix/item-transform-dataset-argument
Sep 12, 2026
Merged

cregouby merged 2 commits into
mlverse:mainfrom
srishtiii28:fix/item-transform-dataset-argument

Conversation

@srishtiii28

@srishtiii28 srishtiii28 commented Sep 9, 2026 •

Copy link
Copy Markdown
Contributor

Closes #399

R CMD check flagged 18 unlockBinding(".getitem", ...) calls, used by the item_transform_*() and target_transform_rotate() dataset methods to patch a dataset's .getitem() in place.

Detection and segmentation datasets now take an item_transform argument,applied to the item after transform and
target_transform. Passing a dataset to item_transform_*() or target_transform_rotate() composes onto that argument instead of unlocking the binding, so the pipe form still works.

@cregouby

Copy link
Copy Markdown
Collaborator

Hello @srishtiii28,

This is a great addition, thank you so much.

@cregouby
cregouby merged commit aae63e2 into mlverse:main Sep 12, 2026
3 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.

Unsafe calls found in code

2 participants