-
Notifications
You must be signed in to change notification settings - Fork 25
Update Jupyter documentation to explain relative paths #220
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
Conversation
|
You can find this (and more) in the OpenSees-on-DesignSafe Training Documentation: |
|
Are you okay with this change here? I think that page is not an appropriate location for such technical information. And @silviamazzoni says that technical information available in training docs that DS User Guide links out too. I've also pinged @mesteva internally, and will carry over her answer if that is where she replies. |
|
Also, whether you use relative or absolute paths depends on your task. |
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.
I got answers.
- @jarosenb, please migrate your improvements / technical information to tools/jupyterhub/#publishing-notebooks.
@jarosenb and @silviamazzoni, please consider whether @silviamazzoni's external-but-related Path documentation should be linked to from tools/jupyterhub/#publishing-notebooks.- Moved to #224.
- After that update, @mesteva will have me update curating/bestpractices/#jupyter-notebooks accordingly to point to that improved content.
|
@wesleyboar , |
3b8e22f to
21636ed
Compare
|
Thanks. Merged. I will add M.E.'s change in a follow-up PR. I will consider S.M.'s suggestions for follow-up improvements. |
This is why #220 was changed, because this content is to be rewritten, non-technical.
|
@silviamazzoni, I've moved your idea to #224.1 Footnotes
|
Encourage users to use relative paths in Jupyter notebooks, with an explanation of the concept and a worked example with some common patterns.