-
Notifications
You must be signed in to change notification settings - Fork 31
fix link #127
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 link #127
Conversation
Click to view CI ResultsGitHub pull request #127 of commit 5e8952d5c15464070585746a08873483e02b3f4a, no merge conflicts.
Running as SYSTEM
Setting status of 5e8952d5c15464070585746a08873483e02b3f4a to PENDING with url https://10.20.13.93:8080/job/merlin_systems/111/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_systems
using credential fce1c729-5d7c-48e8-90cb-b0c314b1076e
> 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/systems # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/systems
> git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems user + githubtoken
> git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/systems +refs/pull/127/*:refs/remotes/origin/pr/127/* # timeout=10
> git rev-parse 5e8952d5c15464070585746a08873483e02b3f4a^{commit} # timeout=10
Checking out Revision 5e8952d5c15464070585746a08873483e02b3f4a (detached)
> git config core.sparsecheckout # timeout=10
> git checkout -f 5e8952d5c15464070585746a08873483e02b3f4a # timeout=10
Commit message: "fix link"
> git rev-list --no-walk 033a5bc565231e389008c27cf7f2787690df687b # timeout=10
[merlin_systems] $ /bin/bash /tmp/jenkins2574688816421798596.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_systems/systems, configfile: pyproject.toml
plugins: anyio-3.5.0, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 14 items / 1 error / 2 skipped
|
Documentation preview |
README.md
Outdated
| ``` | ||
|
|
||
| Refer to the [Merlin Systems Example Notebooks](http://github.io/NVIDIA-Merlin/systems/main/examples/README.html) for a notebook that serves a ranking models ensemble. | ||
| Refer to the [Merlin Systems Example Notebooks](https://github.com/NVIDIA-Merlin/systems/tree/main/examples) for a notebook that serves a ranking models ensemble. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could simplify this link further to ./examples/ if we want the link to be consistent with the commit being viewed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wow, nice one! 🙌 didn't realize you could reference locations on github like this, change made
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The examples directory is a special case because the directory is valuable to folks who browse the repository and to include the notebooks in the documentation. Because the examples/README.md is renamed during the doc build to examples/index.md, we get an examples/index.html and the index file is served by default. Hopefully that doesn't sound too magical.
Looks good and thank you.
Click to view CI ResultsGitHub pull request #127 of commit c06e88c5289b7ee494c9251d032a0a1aff95944f, no merge conflicts.
Running as SYSTEM
Setting status of c06e88c5289b7ee494c9251d032a0a1aff95944f to PENDING with url https://10.20.13.93:8080/job/merlin_systems/112/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_systems
using credential fce1c729-5d7c-48e8-90cb-b0c314b1076e
> 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/systems # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/systems
> git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems user + githubtoken
> git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/systems +refs/pull/127/*:refs/remotes/origin/pr/127/* # timeout=10
> git rev-parse c06e88c5289b7ee494c9251d032a0a1aff95944f^{commit} # timeout=10
Checking out Revision c06e88c5289b7ee494c9251d032a0a1aff95944f (detached)
> git config core.sparsecheckout # timeout=10
> git checkout -f c06e88c5289b7ee494c9251d032a0a1aff95944f # timeout=10
Commit message: "Update README.md"
> git rev-list --no-walk 5e8952d5c15464070585746a08873483e02b3f4a # timeout=10
[merlin_systems] $ /bin/bash /tmp/jenkins8712363033355917160.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_systems/systems, configfile: pyproject.toml
plugins: anyio-3.5.0, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 14 items / 1 error / 2 skipped
|
|
rerun tests |
Click to view CI ResultsGitHub pull request #127 of commit c06e88c5289b7ee494c9251d032a0a1aff95944f, no merge conflicts.
GitHub pull request #127 of commit c06e88c5289b7ee494c9251d032a0a1aff95944f, no merge conflicts.
Running as SYSTEM
Setting status of c06e88c5289b7ee494c9251d032a0a1aff95944f to PENDING with url https://10.20.13.93:8080/job/merlin_systems/113/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_systems
using credential fce1c729-5d7c-48e8-90cb-b0c314b1076e
> 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/systems # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/systems
> git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems user + githubtoken
> git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/systems +refs/pull/127/*:refs/remotes/origin/pr/127/* # timeout=10
> git rev-parse c06e88c5289b7ee494c9251d032a0a1aff95944f^{commit} # timeout=10
Checking out Revision c06e88c5289b7ee494c9251d032a0a1aff95944f (detached)
> git config core.sparsecheckout # timeout=10
> git checkout -f c06e88c5289b7ee494c9251d032a0a1aff95944f # timeout=10
Commit message: "Update README.md"
> git rev-list --no-walk c06e88c5289b7ee494c9251d032a0a1aff95944f # timeout=10
[merlin_systems] $ /bin/bash /tmp/jenkins16741989864396828414.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_systems/systems, configfile: pyproject.toml
plugins: anyio-3.5.0, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 14 items / 1 error / 2 skipped
|
|
rerun tests |
Click to view CI ResultsGitHub pull request #127 of commit c06e88c5289b7ee494c9251d032a0a1aff95944f, no merge conflicts.
GitHub pull request #127 of commit c06e88c5289b7ee494c9251d032a0a1aff95944f, no merge conflicts.
Running as SYSTEM
Setting status of c06e88c5289b7ee494c9251d032a0a1aff95944f to PENDING with url https://10.20.13.93:8080/job/merlin_systems/114/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_systems
using credential fce1c729-5d7c-48e8-90cb-b0c314b1076e
> 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/systems # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/systems
> git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems user + githubtoken
> git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/systems +refs/pull/127/*:refs/remotes/origin/pr/127/* # timeout=10
> git rev-parse c06e88c5289b7ee494c9251d032a0a1aff95944f^{commit} # timeout=10
Checking out Revision c06e88c5289b7ee494c9251d032a0a1aff95944f (detached)
> git config core.sparsecheckout # timeout=10
> git checkout -f c06e88c5289b7ee494c9251d032a0a1aff95944f # timeout=10
Commit message: "Update README.md"
> git rev-list --no-walk c06e88c5289b7ee494c9251d032a0a1aff95944f # timeout=10
[merlin_systems] $ /bin/bash /tmp/jenkins8007024732752815066.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_systems/systems, configfile: pyproject.toml
plugins: anyio-3.5.0, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 14 items / 1 error / 2 skipped
|
|
mhmm so it seems the tests are failing on import of |
|
rerun tests |
Click to view CI ResultsGitHub pull request #127 of commit c06e88c5289b7ee494c9251d032a0a1aff95944f, no merge conflicts.
Running as SYSTEM
Setting status of c06e88c5289b7ee494c9251d032a0a1aff95944f to PENDING with url https://10.20.13.93:8080/job/merlin_systems/115/console and message: 'Pending'
Using context: Jenkins
Building on master in workspace /var/jenkins_home/workspace/merlin_systems
using credential fce1c729-5d7c-48e8-90cb-b0c314b1076e
> 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/systems # timeout=10
Fetching upstream changes from https://github.com/NVIDIA-Merlin/systems
> git --version # timeout=10
using GIT_ASKPASS to set credentials login for merlin-systems user + githubtoken
> git fetch --tags --force --progress -- https://github.com/NVIDIA-Merlin/systems +refs/pull/127/*:refs/remotes/origin/pr/127/* # timeout=10
> git rev-parse c06e88c5289b7ee494c9251d032a0a1aff95944f^{commit} # timeout=10
Checking out Revision c06e88c5289b7ee494c9251d032a0a1aff95944f (detached)
> git config core.sparsecheckout # timeout=10
> git checkout -f c06e88c5289b7ee494c9251d032a0a1aff95944f # timeout=10
Commit message: "Update README.md"
> git rev-list --no-walk c06e88c5289b7ee494c9251d032a0a1aff95944f # timeout=10
[merlin_systems] $ /bin/bash /tmp/jenkins9201396229299800286.sh
============================= test session starts ==============================
platform linux -- Python 3.8.10, pytest-7.1.2, pluggy-1.0.0
rootdir: /var/jenkins_home/workspace/merlin_systems/systems, configfile: pyproject.toml
plugins: anyio-3.5.0, xdist-2.5.0, forked-1.4.0, cov-3.0.0
collected 47 items
|
No description provided.