Skip to content

Update license - #6

Merged
heyufan1995 merged 1 commit into
NVIDIA-Medtech:mainfrom
heyufan1995:main
Mar 2, 2026
Merged

Update license#6
heyufan1995 merged 1 commit into
NVIDIA-Medtech:mainfrom
heyufan1995:main

Conversation

@heyufan1995

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: heyufan <heyufan1995@gmail.com>
@heyufan1995
heyufan1995 merged commit e7eb106 into NVIDIA-Medtech:main Mar 2, 2026
1 check failed
@greptile-apps

greptile-apps Bot commented Mar 2, 2026

Copy link
Copy Markdown

Greptile Summary

Updated license documentation across both NV-Segment-CT and NV-Segment-CTMR models, clarifying that NV-Segment-CT uses a commercial-friendly NVIDIA open model license while NV-Segment-CTMR uses a non-commercial NCLS v1 license.

  • Removed old combined LICENSE files from both model directories
  • Updated VISTA3D citation in NV-Segment-CT README to the published CVPR 2025 version
  • Added License row to main README comparison table with direct links to license documents
  • Fixed 2 minor syntax issues: typo "THe" → "The" and incomplete sentence in CTMR README

Confidence Score: 4/5

  • Safe to merge after fixing the two minor syntax errors
  • Documentation-only changes with clear licensing updates; found two minor syntax issues (typo and incomplete sentence) that should be corrected before merge
  • NV-Segment-CT/docs/README.md and NV-Segment-CTMR/docs/README.md need syntax corrections

Important Files Changed

Filename Overview
NV-Segment-CT/docs/README.md Updated references and license documentation with typo in line 224
NV-Segment-CTMR/docs/README.md Updated license documentation with incomplete sentence on line 349

Last reviewed commit: 79de4db

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

5 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

The model weights included in this project are licensed under the NCLS v1 License.

Both licenses' full texts have been combined into a single `LICENSE` file. Please refer to this `LICENSE` file for more details about the terms and conditions of both licenses.
THe model weights license is under commercial friendly

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Typo: "THe" should be "The"

Suggested change
THe model weights license is under commercial friendly
The model weights license is under commercial friendly

### Model Weights License

The model weights included in this project are licensed under the NCLS v1 License.
The model weights included in this project are licensed under the Non-Commercial

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Incomplete sentence - text cuts off awkwardly before the link

Suggested change
The model weights included in this project are licensed under the Non-Commercial
The model weights included in this project are licensed under the

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.

1 participant