Skip to content

Use local KUI packages#29

Merged
apatole merged 3 commits intonvidia-holoscan:mainfrom
apatole:apatole/use_local_kui
Mar 13, 2025
Merged

Use local KUI packages#29
apatole merged 3 commits intonvidia-holoscan:mainfrom
apatole:apatole/use_local_kui

Conversation

@apatole
Copy link
Collaborator

@apatole apatole commented Feb 26, 2025

To make Holoscan Federated Analytics public, we need to avoid dependency on private repositories. This patch downloads all the KUI packages, stores them in a separate folder and uses it from there instead of private repository.

@apatole apatole force-pushed the apatole/use_local_kui branch 2 times, most recently from 4c28123 to 19702af Compare February 27, 2025 12:01
@apatole apatole self-assigned this Mar 4, 2025
@apatole apatole marked this pull request as draft March 4, 2025 18:20
@apatole apatole force-pushed the apatole/use_local_kui branch from 60511ab to cff3ee5 Compare March 12, 2025 13:45
@apatole apatole marked this pull request as ready for review March 12, 2025 14:04
apatole added 2 commits March 12, 2025 20:27
To make Holoscan Federated Analytics public, we need to avoid
dependency on private repositories. This patch downloads all the KUI
packages, stores them in a separate folder and uses it from there
instead of private repository.

Signed-off-by: Arun Patole <apatole@nvidia.com>
This patch also adds a KUI license text and the license text of its
third party dependencies in 3rdparty folder.

Signed-off-by: Arun Patole <apatole@nvidia.com>
@apatole apatole force-pushed the apatole/use_local_kui branch from cff3ee5 to fd2e79d Compare March 12, 2025 14:57
This change adds a NVIDIA Proprietary copyright header to all the KUI
sources in local npm packages.

Signed-off-by: Arun Patole <apatole@nvidia.com>
@apatole apatole merged commit 60767d3 into nvidia-holoscan:main Mar 13, 2025
2 checks passed
@apatole apatole deleted the apatole/use_local_kui branch March 13, 2025 10:54
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