Skip to content

Conversation

@MMelQin
Copy link
Collaborator

@MMelQin MMelQin commented Mar 1, 2022

Ran spell checker and fixed the typos in the inline comments of all the python files under operators.

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>
Signed-off-by: mmelqin <mingmelvinq@nvidia.com>
@MMelQin MMelQin self-assigned this Mar 1, 2022
@MMelQin MMelQin linked an issue Mar 1, 2022 that may be closed by this pull request
Signed-off-by: mmelqin <mingmelvinq@nvidia.com>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Collaborator

@gigony gigony left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@MMelQin MMelQin merged commit e9a0965 into main Mar 30, 2022
@slbryson
Copy link
Collaborator

Accepted

jlvahldiek pushed a commit to jlvahldiek/monai-deploy-app-sdk that referenced this pull request Mar 31, 2022
* Fixed typo in inline comments.

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>

* Checked and fixed typo for all Operators

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>

* Added notes in the Jupyter notebooks

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>
Signed-off-by: Janis <janis@vahldiek.info>
MMelQin added a commit that referenced this pull request Apr 1, 2022
* Relaxed the restriction on non-DCM files, and raise error based on settings (#273)

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>
Signed-off-by: Janis <janis@vahldiek.info>

* fix pixel spacing

fix pixel spacing if [0x0028, 0x0030] is not available

Signed-off-by: Janis <janis@vahldiek.info>

* fix pixel spacing

fix pixel spacing if [0x0028, 0x0030] is not available

Signed-off-by: Janis <janis@vahldiek.info>

* Revert "fix pixel spacing"

This reverts commit a4073db.

Signed-off-by: Janis <janis@vahldiek.info>

* add default value

add default value of "None" to meta_dict["original_affine"]

Signed-off-by: Janis Vahldiek <janis@vahldiek.info>
Signed-off-by: Janis <janis@vahldiek.info>

* add KeyError

Signed-off-by: Janis Vahldiek <janis@vahldiek.info>
Signed-off-by: Janis <janis@vahldiek.info>

* make black happy

Signed-off-by: Janis <janis@vahldiek.info>

* Mqin/fix doc typos (#274)

* Fixed typo in inline comments.

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>

* Checked and fixed typo for all Operators

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>

* Added notes in the Jupyter notebooks

Signed-off-by: mmelqin <mingmelvinq@nvidia.com>
Signed-off-by: Janis <janis@vahldiek.info>

* Revert "make black happy"

This reverts commit 62996dc.

Signed-off-by: Janis <janis@vahldiek.info>

* make black happy

Signed-off-by: Janis <janis@vahldiek.info>

* fix pixel spacing

fix pixel spacing if [0x0028, 0x0030] is not available

Signed-off-by: Janis <janis@vahldiek.info>

* fix pixel spacing

fix pixel spacing if [0x0028, 0x0030] is not available

Signed-off-by: Janis <janis@vahldiek.info>

* Revert "fix pixel spacing"

This reverts commit a4073db.

Signed-off-by: Janis <janis@vahldiek.info>

* add KeyError

Signed-off-by: Janis Vahldiek <janis@vahldiek.info>

* Revert "make black happy"

This reverts commit 62996dc.

Signed-off-by: Janis <janis@vahldiek.info>

* make black happy

Signed-off-by: Janis <janis@vahldiek.info>

Co-authored-by: Ming M Qin <38891913+MMelQin@users.noreply.github.com>
@MMelQin MMelQin mentioned this pull request Apr 21, 2022
@MMelQin MMelQin deleted the mqin/fix_doc_typos branch February 3, 2025 23:10
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.

[DOC] Misspelling in monai.deploy.operators.MonaiSegInferenceOperator Definition [DOC] Add reminder to restart kernel when installing new packages

4 participants