Skip to content

Conversation

@MMelQin
Copy link
Collaborator

@MMelQin MMelQin commented Jun 4, 2024

The breast density example application was first created with App SDK v0.5. Post SDK v0.6 release, the code had been updated and works when running with Python interpreter. The README file, however, was not updated to correctly use the new MONAI Deploy CLI. There was also an application bug that caused model network loading error when run as an MONAI App Package.

Code and the doc were updated and tested in dev env.

BTW, the Sonar Cloud is complaining about code duplication erroneously, which I have not found a way to acknowledge and mark as such.

@MMelQin MMelQin linked an issue Jun 4, 2024 that may be closed by this pull request
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 4, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
13.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@MMelQin MMelQin merged commit 493c771 into main Jun 5, 2024
@MMelQin MMelQin deleted the mqin/fix_example_breast_density branch February 3, 2025 23:07
SimoneBendazzoli93 pushed a commit to SimoneBendazzoli93/monai-deploy-app-sdk that referenced this pull request Jun 26, 2025
…Project-MONAI#491)

* Fix this example application to make it work with SDK v0.6+

Signed-off-by: M Q <[email protected]>

* Fix the example app errors and corrected folder name

Signed-off-by: M Q <[email protected]>

---------

Signed-off-by: M Q <[email protected]>
Signed-off-by: Simone Bendazzoli <[email protected]>
SimoneBendazzoli93 pushed a commit to SimoneBendazzoli93/monai-deploy-app-sdk that referenced this pull request Jul 17, 2025
…Project-MONAI#491)

* Fix this example application to make it work with SDK v0.6+

Signed-off-by: M Q <[email protected]>

* Fix the example app errors and corrected folder name

Signed-off-by: M Q <[email protected]>

---------

Signed-off-by: M Q <[email protected]>
Signed-off-by: Simone Bendazzoli <[email protected]>
SimoneBendazzoli93 pushed a commit to SimoneBendazzoli93/monai-deploy-app-sdk that referenced this pull request Jul 17, 2025
…Project-MONAI#491)

* Fix this example application to make it work with SDK v0.6+

Signed-off-by: M Q <[email protected]>

* Fix the example app errors and corrected folder name

Signed-off-by: M Q <[email protected]>

---------

Signed-off-by: M Q <[email protected]>
SimoneBendazzoli93 pushed a commit to SimoneBendazzoli93/monai-deploy-app-sdk that referenced this pull request Jul 17, 2025
…Project-MONAI#491)

* Fix this example application to make it work with SDK v0.6+

Signed-off-by: M Q <[email protected]>

* Fix the example app errors and corrected folder name

Signed-off-by: M Q <[email protected]>

---------

Signed-off-by: M Q <[email protected]>
Signed-off-by: Simone Bendazzoli <[email protected]>
SimoneBendazzoli93 pushed a commit to SimoneBendazzoli93/monai-deploy-app-sdk that referenced this pull request Jul 17, 2025
…Project-MONAI#491)

* Fix this example application to make it work with SDK v0.6+

Signed-off-by: M Q <[email protected]>

* Fix the example app errors and corrected folder name

Signed-off-by: M Q <[email protected]>

---------

Signed-off-by: M Q <[email protected]>
Signed-off-by: Simone Bendazzoli <[email protected]>
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] Outdated/incomplete examples documentation

3 participants