-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
docs: initial Jupyter support for website docs, move config notebook #1448
docs: initial Jupyter support for website docs, move config notebook #1448
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1448 +/- ##
=======================================
Coverage 33.19% 33.19%
=======================================
Files 41 41
Lines 4950 4950
Branches 1135 1135
=======================================
Hits 1643 1643
Misses 3177 3177
Partials 130 130
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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
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.
Looks great. One thing I would consider doing is having test in place for the notebooks we have in documentation.
Agreed - @zwd-ms has taken on that task and will be done separately to this PR to ensure it is a holistic solution |
Shall we move it under "Installation"? I think order wise, it's after installation and before use cases. |
I agree, moved it. |
…icrosoft#1448) * docs: jupyter support for website docs, move config docs * update devcontainer dockerfile, fix CI issues * update bsed on working directory * update TLDR * install into temp * remove unrelated change from diff * fix spelling issue * Update sidebars.js
Adds initial support for rendering notebooks using
Quarto
. Quarto is a tool for rendering technical documentation. It has specific support for rendering Jupyter notebooks that target the Docsaurus documentation system so it is particularly helpful for us.Additionally, in this PR I moved the oai_openai_utils notebooks to the main docs section as it is a great notebook for explaining how to setup the LLM config list. I made a small edit to add a
TL;DR
section at the top of the notebook to get started quickly.Sidebar with new page:
Example of rendered notebook:
Related issue number
#1339