Update licensing information in README - #115
Conversation
Clarified licensing terms for NVIDIA Omniverse and other dependencies. Signed-off-by: Stephen R. Aylward <stephen@aylward.org>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Cache: Disabled due to data retention organization setting Knowledge base: Disabled due to data retention organization setting WalkthroughThe README now separates NVIDIA Omniverse licensing from optional non-commercial licenses. It documents license-key requirements for NVIDIA Segment CT MRI weights and TotalSegmentator research-only models, including the ChangesLicensing documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the README’s License section to clarify licensing considerations for NVIDIA Omniverse and other optional dependencies used by PhysioTwin4D.
Changes:
- Reworded the README license text related to NVIDIA Omniverse and added a direct link to the Omniverse license agreement.
- Added a “Non-commercial Licenses (optional)” subsection covering NV-Segment-CTMR weight restrictions and TotalSegmentator research-only model licensing behavior.
Suppressed comments (1)
README.md:143
- The TotalSegmentator bullet wraps onto a new line without indentation and uses triple backticks for inline code, which can break list formatting. Indent continuation lines and use single-backtick inline code spans for method/class references.
* TotalSegmentator includes the optional use of some of their research-only models. Using those models assumes that you have
the appropriate license key install, otherwise an error occurs. Those models can be disabled by calling ```set_has_academic_license(False)``` member function of the ```SegmentChestTotalSegmentator``` class.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Additionally, NVIDIA Omniverse is distributed under its own custom license, which makes it | ||
| free for academic and commercial use. https://docs.omniverse.nvidia.com/ov/latest/common/NVIDIA_Omniverse_License_Agreement.html |
| * NVIDIA Segment CT MRI AI weights (used in the SegmentNVSegmentCTMRI class, | ||
| are restricted from commercial use. https://github.com/NVIDIA-Medtech/NV-Segment-CTMR |
Clarified licensing terms for NVIDIA Omniverse and other dependencies.
Summary by CodeRabbit