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 typo in title for Merlin TensorFlow container #421

Closed

Conversation

mikemckiernan
Copy link
Member

No description provided.

@mikemckiernan mikemckiernan self-assigned this Jun 29, 2022
@mikemckiernan mikemckiernan added the documentation Improvements or additions to documentation label Jun 29, 2022
@mikemckiernan mikemckiernan added this to the Merlin 22.07 milestone Jun 29, 2022
@github-actions
Copy link

Documentation preview

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

@nvidia-merlin-bot
Copy link
Contributor

Click to view CI Results
GitHub pull request #421 of commit 275af5b67c86006596a4fdf8c3571fdc5b434906, no merge conflicts.
Running as SYSTEM
Setting status of 275af5b67c86006596a4fdf8c3571fdc5b434906 to PENDING with url https://10.20.13.93:8080/job/merlin_merlin/208/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/421/*:refs/remotes/origin/pr/421/* # timeout=10
 > git rev-parse 275af5b67c86006596a4fdf8c3571fdc5b434906^{commit} # timeout=10
Checking out Revision 275af5b67c86006596a4fdf8c3571fdc5b434906 (detached)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 275af5b67c86006596a4fdf8c3571fdc5b434906 # timeout=10
Commit message: "Fix typo in title for Merlin TensorFlow container"
 > git rev-list --no-walk 726920d68e44235e299762efaaa197c84d54c8c5 # timeout=10
[merlin_merlin] $ /bin/bash /tmp/jenkins18183016587366855858.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 2 items

tests/unit/test_version.py . [ 50%]
tests/unit/examples/test_building_deploying_multi_stage_RecSys.py . [100%]

======================== 2 passed in 132.13s (0:02:12) =========================
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/jenkins14518299593828952595.sh

@mikemckiernan
Copy link
Member Author

Closing. I added these changes to #426.

@mikemckiernan mikemckiernan deleted the docs-typo-cont-name branch July 5, 2022 17:23
mikemckiernan added a commit that referenced this pull request Jul 6, 2022
* Remove fictional 22.05 container information.
* Add true 22.06 container information.
* Fix typo from #421
mikemckiernan added a commit that referenced this pull request Jul 6, 2022
* Remove fictional 22.05 containers
* Add three new containers for 22.06
* Typo fix for #421
mikemckiernan added a commit that referenced this pull request Jul 7, 2022
* Updates from containers

* Updates to containers for 22.06

* Remove fictional 22.05 containers
* Add three new containers for 22.06
* Typo fix for #421

Co-authored-by: Mike McKiernan <mmckiernan@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants