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.

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

Install hps backend and hps trt plugin in hugectr/tf/pytorch #783

Merged
merged 34 commits into from
Mar 6, 2023

Conversation

EmmaQiaoCh
Copy link
Contributor

@EmmaQiaoCh EmmaQiaoCh commented Jan 4, 2023

  1. Install common dependencies in merlin-base;
  2. Install related components(hugectr inference, hps_backend, hps trt plugin) in merlin-base, merlin-hugectr/merlin-tf/merlin-pytorch all need it;
  3. copy arrow and parquet headers and cmake folder;
  4. specified tritonserverclient to an older version and copy the perf_* binaries which is needed by hugectr

@github-actions
Copy link

github-actions bot commented Jan 4, 2023

Documentation preview

https://nvidia-merlin.github.io/Merlin/review/pr-783

@karlhigley karlhigley added chore Infrastructure update ci labels Jan 4, 2023
@karlhigley
Copy link
Contributor

This looks good. My only comment here is that it looks like this now gets installed in all of the containers, so it probably makes sense to move the install to the base container so we only have to write and maintain it in one place.

@EmmaQiaoCh EmmaQiaoCh changed the title Install hps backend and hps trt plugin in hugectr/tf/pytorch Install hps backend and hps trt plugin in hugectr/tf/pytorch(Pls don't merge it before hugectr 23.01 pushed to github, thanks!) Jan 6, 2023
@karlhigley karlhigley added this to the Merlin 23.02 milestone Jan 31, 2023
@viswa-nvidia
Copy link

@jperez999 to test and report on the size of the container

docker/dockerfile.merlin Outdated Show resolved Hide resolved
@EmmaQiaoCh EmmaQiaoCh changed the title Install hps backend and hps trt plugin in hugectr/tf/pytorch(Pls don't merge it before hugectr 23.01 pushed to github, thanks!) Install hps backend and hps trt plugin in hugectr/tf/pytorch Feb 22, 2023
@jperez999
Copy link
Collaborator

forcing this through for container release.

@jperez999 jperez999 merged commit fd6bef1 into NVIDIA-Merlin:main Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Infrastructure update ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants