-
Notifications
You must be signed in to change notification settings - Fork 24
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
Links to documentation in CohortTracker Tutorial with nbsphinx #881
Conversation
docs/conf.py
Outdated
@@ -8,6 +8,7 @@ | |||
|
|||
HERE = Path(__file__).parent | |||
sys.path[:0] = [str(HERE.parent), str(HERE / "extensions")] | |||
sys.path.append(str(HERE / "tutorials" / "notebooks")) |
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.
what’s this for? sys.path
is for importing python packages, why would you want to do that from the notebook folder?
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.
A leftover thing I wanted to delete again!
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.
Will remove
OK, so you’re using |
OK, got it, the issue is the |
Awesome, this works! To summarize what @flying-sheep did here: Should look like this:
|
Yeah! nbsphinx has docs for this, but they assume you can just do that from UI and don’t have to manually hack around in JSON: https://nbsphinx.readthedocs.io/en/0.2.7/raw-cells.html I filed microsoft/vscode-jupyter#16490 for the ability to do it in VS Code |
test rtd build for nbsphinx link