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 for 'import error' in merlin-tensorflow-training:22.05 #317

Merged
merged 4 commits into from
May 17, 2022

Conversation

EmmaQiaoCh
Copy link
Contributor

Add sok to PYTHONPATH to fix 'import error' when using image 'merlin-tensorflow-training:22.05'

EmmaQiaoCh added 4 commits May 9, 2022 00:47
* change "USE_SSE=1" to "PORTABLE=1" to fix "illegal instruction" error
* To fix import error in merlin-tensorflow-training:22.05
@zehuanw
Copy link
Contributor

zehuanw commented May 17, 2022

Add @jperez999 @benfred @viswa-nvidia to review and merge. Once the hot fix come out @EmmaQiaoCh to test again.

@github-actions
Copy link

Documentation preview

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

@nvidia-merlin-bot
Copy link
Contributor

Click to view CI Results
GitHub pull request #317 of commit 78f9685deabcb4e221c33df0664fc2c9277f42d7, no merge conflicts.
Running as SYSTEM
Setting status of 78f9685deabcb4e221c33df0664fc2c9277f42d7 to PENDING with url https://10.20.13.93:8080/job/merlin_merlin/90/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_merlin
using credential systems-login
 > git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url https://github.com/NVIDIA-Merlin/Merlin # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/Merlin
 > git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems
 > git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/Merlin +refs/pull/317/*:refs/remotes/origin/pr/317/* # timeout=10
 > git rev-parse 78f9685deabcb4e221c33df0664fc2c9277f42d7^{commit} # timeout=10
Checking out Revision 78f9685deabcb4e221c33df0664fc2c9277f42d7 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 78f9685deabcb4e221c33df0664fc2c9277f42d7 # timeout=10
Commit message: "fix: add sok to PYTHONPATH"
 > git rev-list --no-walk a8d252d523eb135e6c8575ea83cd54b2da92b801 # timeout=10
[merlin_merlin] $ /bin/bash /tmp/jenkins2405972752290623522.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_merlin/merlin
plugins: anyio-3.5.0, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 1 item

tests/unit/test_version.py . [100%]

============================== 1 passed in 0.01s ===============================
Performing Post build task...
Match found for : : True
Logical operation result is TRUE
Running script : #!/bin/bash
cd /var/jenkins_home/
CUDA_VISIBLE_DEVICES=1 python test_res_push.py "https://api.GitHub.com/repos/NVIDIA-Merlin/Merlin/issues/$ghprbPullId/comments" "/var/jenkins_home/jobs/$JOB_NAME/builds/$BUILD_NUMBER/log"
[merlin_merlin] $ /bin/bash /tmp/jenkins2982214604134177445.sh

@benfred benfred merged commit 57650bd into NVIDIA-Merlin:main May 17, 2022
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.

4 participants