-
Notifications
You must be signed in to change notification settings - Fork 1.5k
There are 2 kinds of random state in transforms #3124
Copy link
Copy link
Closed
Labels
bugSomething isn't workingSomething isn't workingquestionFurther information is requestedFurther information is requested
Milestone
Description
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:
- Use the self.R of first transform the second:
https://github.com/Project-MONAI/MONAI/blob/dev/monai/transforms/croppad/dictionary.py#L761 - 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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingquestionFurther information is requestedFurther information is requested