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

Fix libnvinfer conflict #855

Merged
merged 7 commits into from
Mar 22, 2023

Conversation

EmmaQiaoCh
Copy link
Contributor

  1. remove the specified version of libnvinfer
  2. copy libcupti.so.12 instead of libcupti.so.11.8 which is not in upstream container
  3. update TFDE_VER=v0.3 to fix some build error

@github-actions
Copy link

Documentation preview

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

@karlhigley karlhigley added chore Infrastructure update ci labels Mar 14, 2023
@karlhigley karlhigley added this to the Merlin 23.03 milestone Mar 14, 2023
@karlhigley karlhigley requested a review from jperez999 March 14, 2023 15:57
@jperez999
Copy link
Collaborator

Running this in pre-release environment. Will report back if any issues... otherwise I will approve.

@@ -198,15 +198,15 @@ RUN apt update -y --fix-missing && \
# Required to build Protocol Buffers.
autoconf automake libtool \
# Required to build Hadoop.
default-jdk maven pkg-config \
Copy link
Collaborator

Choose a reason for hiding this comment

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

Love that we can remove this.

@karlhigley
Copy link
Contributor

The test failure here is unrelated and we have an open PR to fix it. Will merge that one and re-run this.

@EmmaQiaoCh
Copy link
Contributor Author

@jperez999 @karlhigley Thanks for your info, but we just hit regression errors when testing SOK and hps_trt on the new TF, so please hold on this PR although your tests passed. We may need more changes.

@karlhigley karlhigley merged commit 3e9593f into NVIDIA-Merlin:main Mar 22, 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.

3 participants