Skip to content
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

Add transform to apply orientation to guidance #1537

Closed
wants to merge 8 commits into from

Conversation

diazandr3s
Copy link
Collaborator

Signed-off-by: Andres [email protected]

Copy link
Collaborator

@wyli wyli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, looks good to me, please note that this only works in 3d.

@Nic-Ma
Copy link

Nic-Ma commented Sep 5, 2023

Hi @diazandr3s ,

@yiheng-wang-nv will update the DeepEdit bundle according to your change here, is it OK? And please help review later.

Thanks in advance.

yiheng-wang-nv added a commit to Project-MONAI/model-zoo that referenced this pull request Sep 8, 2023
Fixes #500 .

### Description
This PR is used to fix the orientation issue of the spleen deepedit
annotation bundle.
This PR refers to Project-MONAI/MONAILabel#1537


### Status
**Ready**

### Please ensure all the checkboxes:
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [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"`).

---------

Signed-off-by: Yiheng Wang <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@SachidanandAlle
Copy link
Collaborator

Do we need this..?

@YanxuanLiu
Copy link
Collaborator

/build

1 similar comment
@YanxuanLiu
Copy link
Collaborator

/build

@bastiao
Copy link

bastiao commented Jul 5, 2024

On our side, we check and it seems that we need this! Thanks for fixing it.

yiheng-wang-nv added a commit to yiheng-wang-nv/model-zoo that referenced this pull request Jul 29, 2024
Fixes Project-MONAI#500 .

### Description
This PR is used to fix the orientation issue of the spleen deepedit
annotation bundle.
This PR refers to Project-MONAI/MONAILabel#1537


### Status
**Ready**

### Please ensure all the checkboxes:
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [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"`).

---------

Signed-off-by: Yiheng Wang <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@SachidanandAlle
Copy link
Collaborator

Closing this due to no activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants