Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update to Triton Inference Server container version 24.09 #2001

Merged
merged 17 commits into from
Oct 31, 2024
Merged

Update to Triton Inference Server container version 24.09 #2001

merged 17 commits into from
Oct 31, 2024

Conversation

dagardner-nv
Copy link
Contributor

@dagardner-nv dagardner-nv commented Oct 28, 2024 •
edited
Loading

Description

Closes #2000

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • When the PR is ready for review, new or existing tests cover these changes.
  • When the PR is ready for review, the documentation is up to date with these changes.

@dagardner-nv dagardner-nv requested review from a team as code owners October 28, 2024 16:45
@dagardner-nv dagardner-nv self-assigned this Oct 28, 2024
@dagardner-nv dagardner-nv changed the base branch from branch-25.02 to branch-24.10 October 28, 2024 16:46
@dagardner-nv dagardner-nv marked this pull request as draft October 28, 2024 16:46
@dagardner-nv dagardner-nv added non-breaking Non-breaking change improvement Improvement to existing functionality labels Oct 28, 2024
dagardner-nv and others added 14 commits October 28, 2024 15:06
@dagardner-nv
Fixes bug when taking a slice of a slice
2386849
@dagardner-nv
Merge branch 'david-slice-2002' into david-update-triton-24.09 [no-ci]
d92cb29
@dagardner-nv
Simple repro test for Morpheus issue nv-morpheus#2002
c7d23b4
@dagardner-nv
Give the test a better name, confirm the issue is specific to get_data
163ab38
@dagardner-nv
Assert we are receving slices
6335a8b
@dagardner-nv
Avoid some redundant code
b352f0a
@dagardner-nv
Create copies rather than slices, remove unused make_output_message m…
7f3a9eb 
…ethod
@dagardner-nv
IWYI
3e61fef
@dagardner-nv
Add comment
3187eb1
@dagardner-nv
Merge branch 'david-set_meta-2011' into david-update-triton-24.09
419fdee
@dagardner-nv
Fix output dims
243bd6a
@tzemicheal
Update ransomware model file
420ef7a
@dagardner-nv
Merge branch 'branch-24.10' of github.com:nv-morpheus/Morpheus into d…
bf0beb4 
…avid-update-triton-24.09
@dagardner-nv
Merge branch 'tz-ransomware-modelfile' of github.com:tzemicheal/Morph…
2f8d440 
…eus into david-update-triton-24.09
@copy-pr-bot copy-pr-bot
Copy link

copy-pr-bot bot commented Oct 30, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@dagardner-nv
Copy link
Contributor Author

/ok to test

@dagardner-nv dagardner-nv marked this pull request as ready for review October 30, 2024 23:11
cwharris
cwharris reviewed Oct 30, 2024
View reviewed changes
Copy link
Contributor

@cwharris cwharris left a comment

Choose a reason for hiding this comment

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

LGTM

raykallen
raykallen approved these changes Oct 30, 2024
View reviewed changes
Copy link
Contributor

@raykallen raykallen left a comment

Choose a reason for hiding this comment

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

lgtm

@dagardner-nv
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit f5d8191 into nv-morpheus:branch-24.10 Oct 31, 2024
17 of 18 checks passed
@dagardner-nv dagardner-nv mentioned this pull request Oct 31, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdemoret-nv mdemoret-nv mdemoret-nv approved these changes

@raykallen raykallen raykallen approved these changes

@cwharris cwharris cwharris approved these changes

Assignees

@dagardner-nv dagardner-nv

Labels
improvement Improvement to existing functionality non-breaking Non-breaking change
Projects
Morpheus Boards
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEA]: Investigate Triton server update
5 participants
@dagardner-nv @cwharris @raykallen @mdemoret-nv @tzemicheal