docs: add juptyer API tutorial for API and jupyter tutorial index#3
Merged
Merged
Conversation
…ng edited when we are done
317brian
marked this pull request as ready for review
November 1, 2022 22:46
techdocsmith
reviewed
Nov 1, 2022
techdocsmith
left a comment
Collaborator
There was a problem hiding this comment.
Couple comments. But otherwise this is what I was thinking.
|
|
||
| - An available Druid instance. You can use the `micro-quickstart` configuration described in [Quickstart (local)](./index.md). The tutorials assume that you are using the quickstart, so no authentication or authorization is expected unless explicitly mentioned. | ||
|
|
||
| - The `requests` package for Python. For example, you can install it with the following command: |
| ~ under the License. | ||
| --> | ||
|
|
||
| There are Jupyter notebook-based tutorials that are available for you to try as you learn more about Druid. These tutorials provide snippets of Python code that you can use to run calls against the Druid API. |
Collaborator
There was a problem hiding this comment.
Suggested change
| There are Jupyter notebook-based tutorials that are available for you to try as you learn more about Druid. These tutorials provide snippets of Python code that you can use to run calls against the Druid API. | |
| There are Jupyter notebook-based tutorials that are available for you to try as you learn more about Druid. These tutorials provide snippets of Python code that you can use to run calls against the Druid API. |
Collaborator
There was a problem hiding this comment.
Something I don't like about "There are". What about: You can try out the APIs using the notebooks... or something.
317brian
added a commit
that referenced
this pull request
Nov 9, 2022
(cherry picked from commit aeb8d9e)
317brian
added a commit
that referenced
this pull request
Feb 22, 2023
* docs: add juptyer API tutorial for API and jupyter tutorial index (#3) (cherry picked from commit aeb8d9e) * update prereqs and fix jupyterlab name * Removing notebook since 13345 has it 13345 should be merged first * update contributing instructions * docs: link to the Druid SQL tutorial * Add link to partitioning * fix merge conflict * Saving * Update docs/tutorials/tutorial-jupyter-index.md * Remove partitioning --------- Co-authored-by: 317brian <[email protected]> Co-authored-by: brian.le <[email protected]> Co-authored-by: Charles Smith <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
assetsfolder. This shouldn't be an issue since it's not much bigger than a markdown file of equivalent length.After this is reviewed/approved, I will make a commit that locks the markdown cells to be uneditable by default. (This is more a guardrail for accidents. Someone still could if they change the metadata).