Mq/release 3.3 #567
Merged
Mq/release 3.3 #567
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: M Q <mingmelvinq@nvidia.com>
Signed-off-by: M Q <mingmelvinq@nvidia.com>
Signed-off-by: M Q <mingmelvinq@nvidia.com>
|
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR bumps the MONAI Deploy App SDK version to 3.3.0 and updates documentation dependencies to newer, more flexible versions. The release adds compatibility with MONAI v1.5.1, Holoscan v3.7 CUDA12, and fixes volumetric DICOM conversion metadata consistency.
- Version bumped from 3.2.0 to 3.3.0
- Sphinx and related documentation dependencies updated to use version ranges instead of pinned versions
recommonmarkdependency removed (deprecated package)
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| requirements-dev.txt | Updated Sphinx, sphinx-autodoc-typehints, and sphinx-rtd-theme to flexible version ranges; removed deprecated recommonmark dependency |
| docs/source/release_notes/v3.3.0.md | Added release notes for version 3.3.0 with MONAI v1.5.1 and Holoscan v3.7 CUDA12 compatibility |
| docs/source/release_notes/index.md | Added v3.3.0 to the release notes table of contents |
| .bumpversion.cfg | Updated current_version from 3.2.0 to 3.3.0 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ### Key changes | ||
|
|
||
| - Compatible with the latest MONAI v1.5.1 and Holoscan v3.7 CUDA12 version, [holoscan-cu12](https://pypi.org/project/holoscan-cu12). | ||
| - Volumentric image converted from DICOM has consistent `affine` and `space` metadata, e.g. converted image in original DICOM orientation is in `LPS`. |
There was a problem hiding this comment.
Corrected spelling of 'Volumentric' to 'Volumetric'.
Suggested change
| - Volumentric image converted from DICOM has consistent `affine` and `space` metadata, e.g. converted image in original DICOM orientation is in `LPS`. | |
| - Volumetric image converted from DICOM has consistent `affine` and `space` metadata, e.g. converted image in original DICOM orientation is in `LPS`. |
MMelQin
added a commit
that referenced
this pull request
Oct 31, 2025
* Update release notes for v3.3.0 Signed-off-by: M Q <mingmelvinq@nvidia.com> * Update the index file Signed-off-by: M Q <mingmelvinq@nvidia.com> * Bump version: 3.2.0 → 3.3.0 Signed-off-by: M Q <mingmelvinq@nvidia.com> --------- Signed-off-by: M Q <mingmelvinq@nvidia.com>
MMelQin
added a commit
that referenced
this pull request
Oct 31, 2025
) * Initial code for making a RESTful miscroservice with MONAI App SDK Signed-off-by: M Q <mingmelvinq@nvidia.com> * Removed operator local fix and use newly released MONAI App SDK v3.5.1 release Signed-off-by: M Q <mingmelvinq@nvidia.com> * Removed local copy of operator that had a temp fix Signed-off-by: M Q <mingmelvinq@nvidia.com> * Add proper callback message Signed-off-by: M Q <mingmelvinq@nvidia.com> * Proper json string and security fix Signed-off-by: M Q <mingmelvinq@nvidia.com> * Fix complaints Signed-off-by: M Q <mingmelvinq@nvidia.com> * Updated readme Signed-off-by: M Q <mingmelvinq@nvidia.com> * Update per review comments Signed-off-by: M Q <mingmelvinq@nvidia.com> * More update Signed-off-by: M Q <mingmelvinq@nvidia.com> * Set header Signed-off-by: M Q <mingmelvinq@nvidia.com> * Added packaging and running MAP of the REST service Signed-off-by: M Q <mingmelvinq@nvidia.com> * Add app.yaml that has missed the checkin Signed-off-by: M Q <mingmelvinq@nvidia.com> * pin monai<=1.5.0 for now as 1.5.1 has breaking changes Signed-off-by: M Q <mingmelvinq@nvidia.com> * Add CCHMC nnUNet fifteen checkpoint application example Signed-off-by: Elan Somasundaram <chezhipower@gmail.com> Signed-off-by: chezhia <chezhipower@gmail.com> * Update README documentation Signed-off-by: Elan Somasundaram <chezhipower@gmail.com> Signed-off-by: chezhia <chezhipower@gmail.com> * flake8 fixed Signed-off-by: chezhia <chezhipower@gmail.com> * all tests pass Signed-off-by: chezhia <chezhipower@gmail.com> * Make affine and space metadata consistent as well as updating support of latest holoscan SDK CUDA 12 version (#565) * Make affine and space consistent as space is properly parse by MONAI transforms Signed-off-by: M Q <mingmelvinq@nvidia.com> * Typing improvements Signed-off-by: M Q <mingmelvinq@nvidia.com> * Make image metadata Affine and Space consistent, either LPS or RAS Signed-off-by: M Q <mingmelvinq@nvidia.com> * Update monai/deploy/operators/monai_seg_inference_operator.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Ming M Qin <38891913+MMelQin@users.noreply.github.com> * Correct CoPilot suggested code which failed liniting Signed-off-by: M Q <mingmelvinq@nvidia.com> * Fix complaint from new version of mypy Signed-off-by: M Q <mingmelvinq@nvidia.com> * Support the latest holoscan SDK CUDA 12 version, holoscan-cu12 Signed-off-by: M Q <mingmelvinq@nvidia.com> * Docs gen works with Python 3.10+ Signed-off-by: M Q <mingmelvinq@nvidia.com> * Fix complaints on single quote vs dhouble quote for string Signed-off-by: M Q <mingmelvinq@nvidia.com> * Making docs gen require python >= 3.10 Signed-off-by: M Q <mingmelvinq@nvidia.com> * Fix docs build error on readthedocs, although local builds had no issues Signed-off-by: M Q <mingmelvinq@nvidia.com> --------- Signed-off-by: M Q <mingmelvinq@nvidia.com> Signed-off-by: Ming M Qin <38891913+MMelQin@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Mq/release 3.3 (#567) * Update release notes for v3.3.0 Signed-off-by: M Q <mingmelvinq@nvidia.com> * Update the index file Signed-off-by: M Q <mingmelvinq@nvidia.com> * Bump version: 3.2.0 → 3.3.0 Signed-off-by: M Q <mingmelvinq@nvidia.com> --------- Signed-off-by: M Q <mingmelvinq@nvidia.com> * Use string enum for status Signed-off-by: M Q <mingmelvinq@nvidia.com> * Fix the lovely formatting complaint Signed-off-by: M Q <mingmelvinq@nvidia.com> --------- Signed-off-by: M Q <mingmelvinq@nvidia.com> Signed-off-by: Elan Somasundaram <chezhipower@gmail.com> Signed-off-by: chezhia <chezhipower@gmail.com> Signed-off-by: Ming M Qin <38891913+MMelQin@users.noreply.github.com> Co-authored-by: Elan Somasundaram <chezhipower@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.