update ldm noise scheduler api for inference

[Can-Zhao]

Fixes # .

Description

update ldm noise scheduler api for inference

Status

Ready/Work in progress/Hold

Please ensure all the checkboxes:

• [x] Codeformat tests passed locally by running ./runtests.sh --codeformat.
• [ ] In-line docstrings updated.
• [ ] Update version and changelog in metadata.json if changing an existing bundle.
• [ ] Please ensure the naming rules in config files meet our requirements (please refer to: CONTRIBUTING.md).
• [ ] Ensure versions of packages such as monai, pytorch and numpy are correct in metadata.json.
• [ ] Descriptions should be consistent with the content, such as eval_metrics of the provided weights and TorchScript modules.
• [ ] Files larger than 25MB are excluded and replaced by providing download links in large_file.yml.
• [ ] Avoid using path that contains personal information within config files (such as use /home/your_name/ for "bundle_root").

[yiheng-wang-nv]

Thanks @Can-Zhao , I will add the unit tests on these 2 bundles within this PR to prevent similar issues.

[yiheng-wang-nv]

Hi @Nic-Ma , when preparing the unit test of ldm bundles, I just found a place of ldm bundles that is not matched up with: https://github.com/Project-MONAI/MONAI/blob/e6ec945e4b87b90835cdad29ea64b1f27b8accda/monai/bundle/properties.py#L61

Some datasets may not have "data", such as all datasets in: https://github.com/Project-MONAI/MONAI/blob/dev/monai/apps/datasets.py or customized datasets.

Is it possible to change it into BundleProperty.REQUIRED: False ? Or ldm bundles should extend BundleWorkflow directly and define a customized workflow (within the unit test) ?

[Nic-Ma]

Hi @yiheng-wang-nv ,

That's a good point, I would suggest to change the required train_dataset_data into BundleProperty.REQUIRED: False in the properties. CC @SachidanandAlle and @tangy5 .

Thanks.

[yiheng-wang-nv]

Hi @yiheng-wang-nv ,

That's a good point, I would suggest to change the required train_dataset_data into BundleProperty.REQUIRED: False in the properties. CC @SachidanandAlle and @tangy5 .

Thanks.

I see. Hi @SachidanandAlle @tangy5 , do you think we can change the "BundleProperty.REQUIRED" of train_dataset_data? Are there any concerns here?

[YanxuanLiu]

/build

[yiheng-wang-nv]

/build