-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add documentation that exists on the Wiki page also in the doc folder #179
Conversation
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.
/lgtm
/approve
thanks 💯
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: harshad16 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest-required |
* Add the documentation that is on the Wiki page also in the doc folder (#179) * Add documentation for recommended accelerators annotation (#177) * Add test cases for the pytorch notebooks * Add test notebook suite for tensorflow notebook Add test cases for the tensorflow notebook * docs(readme): fix typo in image names * updates to makefile and readme, plus fixed wrong python version * Fix UBI version used on particular places with regards the Pyton version * Use RHEL9 repository for UBI9 and CentOs Stream 9 Cuda images * Add tests for minimal notebooks Signed-off-by: rkpattnaik780 <rkpattnaik780@gmail.com> * Add tests for datascience notebooks Signed-off-by: rkpattnaik780 <rkpattnaik780@gmail.com> * updates to makefile and readme * upgrade trustyai notebook images to version 0.3.0 (#209) * Add test case to convert a pytorch model to onnx format (#213) * Fix of UBI and Python version used for Jupyter TrustyAI image (#224) This is a similar fix as the #199 was. Was brought in via #164 that was merged later. * Set correctly for redirects on vscode and rstudio nginx (#226) * adding elyra to imagestream (#222) --------- Signed-off-by: rkpattnaik780 <rkpattnaik780@gmail.com> Co-authored-by: aTheo <atheodor@redhat.com> Co-authored-by: Ramakrishna Pattnaik <rkpattnaik780@gmail.com> Co-authored-by: atheo89 <atheodorak@outlook.com> Co-authored-by: Tommaso Teofili <tteofili@redhat.com> Co-authored-by: Jan Stourac <jstourac@redhat.com> Co-authored-by: Diamond Bryant <135062655+dibryant@users.noreply.github.com>
…mMain [fix] hadolint warnings to base and rstudio Dockerfiles
…o-release-2024a [release-2024a] Cherry pick opendatahub-io#179, opendatahub-io#253 and opendatahub-io#254 to release-2024a
Add the documentation that is on the Wiki page also in the doc folder.
Description
Since all of the rest repos keep the documentation in a particular folder on the repo we want to do the same.
Related to:
Documentation about the Workbenches and Architecture of the repo. #160
Documentation about the getting started and contribution guideline for notebooks repo. #161
Documentation on Selecting the workbench images from the dashboard. #162
The commits are squashed in a cohesive manner and have meaningful messages.
Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
The developer has manually tested the changes and verified that the changes work