Conversation
Click to view CI ResultsGitHub pull request #498 of commit 10716bfb800e562c74945a34acaa4cbddaad1552, no merge conflicts.
Running as SYSTEM
Setting status of 10716bfb800e562c74945a34acaa4cbddaad1552 to PENDING with url http://10.20.17.181:8080/job/transformers4rec_tests/202/ 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/498/*:refs/remotes/origin/pr/498/* # timeout=10
> git rev-parse 10716bfb800e562c74945a34acaa4cbddaad1552^{commit} # timeout=10
Checking out Revision 10716bfb800e562c74945a34acaa4cbddaad1552 (detached)
> git config core.sparsecheckout # timeout=10
> git checkout -f 10716bfb800e562c74945a34acaa4cbddaad1552 # timeout=10
Commit message: "set ignore masking to True by default"
> git rev-list --no-walk 7e7e5e3e90d0c972e80303dc8f4422e132ac6b2a # timeout=10
[transformers4rec_tests] $ /bin/bash /tmp/jenkins13522023760786896743.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-2.5.0, forked-1.4.0, cov-4.0.0
collected 1 item
|
Documentation previewhttps://nvidia-merlin.github.io/Transformers4Rec/review/pr-498 |
| outputs = {} | ||
| if forward: | ||
| predictions = self(predictions) | ||
| predictions = self(predictions, ignore_masking=False) |
There was a problem hiding this comment.
To compute metrics and losses we need to compute the masked targets. So we should not ignore masking
Click to view CI ResultsGitHub pull request #498 of commit 6be97b9272bd97ff4576309aa517725f7343340d, no merge conflicts.
Running as SYSTEM
Setting status of 6be97b9272bd97ff4576309aa517725f7343340d to PENDING with url http://10.20.17.181:8080/job/transformers4rec_tests/225/ 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/498/*:refs/remotes/origin/pr/498/* # timeout=10
> git rev-parse 6be97b9272bd97ff4576309aa517725f7343340d^{commit} # timeout=10
Checking out Revision 6be97b9272bd97ff4576309aa517725f7343340d (detached)
> git config core.sparsecheckout # timeout=10
> git checkout -f 6be97b9272bd97ff4576309aa517725f7343340d # timeout=10
Commit message: "Merge branch 'main' into ignore-masking"
> git rev-list --no-walk 0f5b64bf82ddf0857da7b74f504fa92238d6112f # timeout=10
[transformers4rec_tests] $ /bin/bash /tmp/jenkins16112933851064363462.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-2.5.0, forked-1.4.0, cov-4.0.0
collected 1 item
|
|
Looks good from a serving standpoint 👍🏻 |
Fixes NVIDIA-Merlin/NVTabular#1688
ignore_maskingto True in the model's forward method.