/build

Hi @wyli @SachidanandAlle @rijobro @KumoLiu ,

Instead of permanently reseting the op ids in the applied_transforms, I would suggest to:

1. Provide a module flag with the set/get APIs to enable / disable the ID checking, similar to our MetaTensor:
   https://github.com/Project-MONAI/MONAI/blob/dev/monai/data/meta_obj.py#L30
2. Then update the default value of check arg to check=get_id_check():

def pop_transform(self, data, key: Hashable = None, check: bool = get_id_check())

https://github.com/Project-MONAI/MONAI/blob/dev/monai/transforms/inverse.py#L191

The benefit is that:

1. The applied_transform information is still there.
2. If some spetial transform must work with the original instance, the developer can explicitly set pop_transform(check=True) in its implementation.

What do you guys think?
Maybe I misunderstood the initial feature request, open for discussion.

Thanks.

Hi @Nic-Ma in my understanding when we store the applied operations info in a persistent cache, the id info becomes invalid because the cache is supposed to be loaded in other runs. In this case resetting the ids would be good. I'd be reluctant to change the pop_transform API unless we see a good use case for it. Note for some transforms there's also an 'inverse_transform' method to 'bypass' the check

Hi @wyli ,

I think users usually use persistent dataset for 2 reasons:
(1) Cache in the persistent storage, then reuse it in other runs.
(2) Cache in the persistent storage due to leak of memory storage in the typical training, then every epoch can reuse it.

For (1) you are right, the info becomes invalid, for (2) the applied transforms are still the original instances.
May I know what's the drawback of my module flag idea?
@rijobro @ericspod What do you guys think?

Thanks.

I think setting the data id would have finer control of the check. There's nothing wrong with your global flag, it might be useful in the future to completely turn off the check, but it's not needed at the moment. I think we should avoid global flags whenever possible.

/build