-
Notifications
You must be signed in to change notification settings - Fork 342
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update auto3dseg algorithm templates #184
Update auto3dseg algorithm templates #184
Conversation
auto3dseg/algorithm_templates/dints/configs/hyper_parameters.yaml
Outdated
Show resolved
Hide resolved
I would like to hear @Nic-Ma 's thought on the |
Hi @dongyang0122 @mingxin-zheng , I tried to reproduce the issue with below config: {
"data": [1, 2],
"data_per_batch": 2,
"transform": {
"_target_": "Compose",
"transforms": [
{"_target_": "LoadImaged", "keys": "image"},
{"_target_": "RandTorchVisiond", "keys": "@##0#keys", "name": "ColorJitter", "brightness": 0.25},
],
},
"dataset": {"_target_": "Dataset", "data": "@data", "transform": "@transform"},
"dataloader": {
"_target_": "DataLoader",
"dataset": "@##dataset",
"batch_size": "@data_per_batch",
"collate_fn": "$monai.data.list_data_collate",
},
} But seems the first 2 lines can be parsed OK: "data": [1, 2],
"data_per_batch": 2 Could you please help provide a simple program to reproduce the issue you asked? Thanks. |
Hi @dongyang0122 , if the fix in the |
If we convert it back to previous name, then we have to inform users to install monai from the dev branch. I feel it adds another layer of complexity for general users. how about converting it back once the changes are reflected in the installation via |
Signed-off-by: Mingxin Zheng <18563433+mingxin-zheng@users.noreply.github.com> Fixes #5972 . ### Description Fix and New Feature included: - fix cache rate: Project-MONAI/research-contributions#173 - typo fix Project-MONAI/research-contributions#178 - scheduler step fix Project-MONAI/research-contributions#182 - enhancement Project-MONAI/research-contributions#184 - Fixed pretrain weight loading Project-MONAI/research-contributions#202 ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [x] `python tests/test_integration_autorunner.py` succeeded locally --------- Signed-off-by: Mingxin Zheng <18563433+mingxin-zheng@users.noreply.github.com>
Update auto3dseg algorithm templates for performance (accuracy) and efficiency improvement.