Skip to content

There are 2 kinds of random state in transforms #3124

@Nic-Ma

Description

@Nic-Ma

Describe the bug
Currently, in the case that a random transform contains another random transform, we have 2 different ways to set the random status:

  1. Use the self.R of first transform the second:
    https://github.com/Project-MONAI/MONAI/blob/dev/monai/transforms/croppad/dictionary.py#L761
  2. Use the same random seed or state for both transforms:
    https://github.com/Project-MONAI/MONAI/blob/dev/monai/transforms/intensity/dictionary.py#L177

@wyli @rijobro @ericspod Which one do you guys think is better? Then I can try to unify them.

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingquestionFurther information is requested

Type

No type

Projects

Relationships

None yet

Development

No branches or pull requests

Issue actions