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 name and bug in MeanReciprocalRankAt #522

Merged
merged 1 commit into from
Nov 8, 2022
Merged

Fix name and bug in MeanReciprocalRankAt #522

merged 1 commit into from
Nov 8, 2022

Conversation

rnyak
Copy link
Contributor

@rnyak rnyak commented Nov 8, 2022

Replicates of the #513

@rnyak rnyak requested a review from sararb November 8, 2022 20:31
@rnyak rnyak added the chore Maintenance for the repository label Nov 8, 2022
@nvidia-merlin-bot
Copy link

Click to view CI Results
GitHub pull request #522 of commit 2ac105763e603045679a4aa91e596c70d2ab01f0, no merge conflicts.
Running as SYSTEM
Setting status of 2ac105763e603045679a4aa91e596c70d2ab01f0 to PENDING with url http://merlin-infra1.nvidia.com:8080/job/transformers4rec_tests/249/ and message: 'Build started for merge commit.'
Using context: Jenkins Unit Test Run
Building on master in workspace /var/jenkins_home/workspace/transformers4rec_tests
using credential nvidia-merlin-bot
Cloning the remote Git repository
Cloning repository https://github.com/NVIDIA-Merlin/Transformers4Rec.git
 > git init /var/jenkins_home/workspace/transformers4rec_tests/transformers4rec # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/Transformers4Rec.git
 > git --version # timeout=10
using GIT_ASKPASS to set credentials This is the bot credentials for our CI/CD
 > git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/Transformers4Rec.git +refs/heads/*:refs/remotes/origin/* # timeout=10
 > git config remote.origin.url https://github.com/NVIDIA-Merlin/Transformers4Rec.git # timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # timeout=10
 > git config remote.origin.url https://github.com/NVIDIA-Merlin/Transformers4Rec.git # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/Transformers4Rec.git
using GIT_ASKPASS to set credentials This is the bot credentials for our CI/CD
 > git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/Transformers4Rec.git +refs/pull/522/*:refs/remotes/origin/pr/522/* # timeout=10
 > git rev-parse 2ac105763e603045679a4aa91e596c70d2ab01f0^{commit} # timeout=10
Checking out Revision 2ac105763e603045679a4aa91e596c70d2ab01f0 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 2ac105763e603045679a4aa91e596c70d2ab01f0 # timeout=10
Commit message: "fix typo and metric error"
 > git rev-list --no-walk 93c5bb5707a18286b184ee9a47b8487b71bbaba8 # timeout=10
[transformers4rec_tests] $ /bin/bash /tmp/jenkins10682911197966717553.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.3, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/transformers4rec_tests/transformers4rec
plugins: anyio-3.6.1, xdist-3.0.2, cov-4.0.0
collected 1 item

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

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

@github-actions
Copy link

github-actions bot commented Nov 8, 2022

@rnyak rnyak merged commit 22be4fd into main Nov 8, 2022
@rnyak rnyak deleted the fix_metric_typo branch November 8, 2022 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Maintenance for the repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants