Skip to content
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

Create docs #434

Merged
merged 4 commits into from
Mar 26, 2022
Merged

Create docs #434

merged 4 commits into from
Mar 26, 2022

Conversation

ianhi
Copy link
Collaborator

@ianhi ianhi commented Feb 25, 2022

Scattered over a few issues there has been mention of the need for a documentation page. This is a first pass at creating one. I think there are lots of great things we can add (e.g. css tricks #411 (comment)) but getting something out there is more important than getting everything in there. I think it's important for users to be able to go to a website to get install instructions rather than a github repo as I suspect that this is lends a sense of stability and officiallness to the project. It also makes it easier to have much more robust sections on installing and contributing.

I think the big question I have is probably primarily for @tacaswell (and also @martinRenou ) which is where should this be hosted? I know how to set things up on readthedocs and so can can easily do that. But this is also a matplotlib subproject (unclear how official that makes things) and does that mean that this shoudl be hosted on matplotlib.org/ipympl? I think that would be my preferred URL if possible.

Other thoughts:

  1. One day my dream is to have the docs be immediately enabled as interactive via pyodide @martinRenou do you know how far we are from being able to do something like that?
  2. We should have a more comprehensive section on using ipympl with ipywidgets
    • Thoughts on including mpl-interactions there? I'm biased but I find it to be the best way to pair ipywidgets with matplotlib.

@github-actions
Copy link
Contributor

Binder 👈 Launch a binder notebook on branch ianhi/ipympl/docs

Comment on lines +25 to +28
<!--
## Sagemath CoCalc
TODO
-->
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anyone have an idea if/how to get ipympl working in sagemath/cocalc?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that there's less noise here: @williamstein I couldn't find an obvious way to do this after some brief searching and the one attempt I had didn't work. Would love to include instructions though

@martinRenou
Copy link
Member

One day my dream is to have the docs be immediately enabled as interactive via pyodide @martinRenou do you know how far we are from being able to do something like that?

We're pretty close actually :)

I'd really like to have a simple replite console at the beginning of the docs. But this requires widgets to work in JupyterLab console jupyter-widgets/ipywidgets#3004

@martinRenou
Copy link
Member

I'm making some experiments on jupyterlite-sphinx:

https://github.com/martinRenou/jupyterlite-sphinx
https://jupyterlite-sphinx.readthedocs.io/en/latest

@ianhi
Copy link
Collaborator Author

ianhi commented Mar 1, 2022

bump @tacaswell

@tacaswell
Copy link
Member

I'm sorry, I wrote message here (and even checked it off my to-do list) but apparently never hit "send" 🐑 🤦🏻

I think be best path to hosting is to use something like https://github.com/matplotlib/mpl-gui/blob/main/.github/workflows/docs.yml to put the built docs on github. We just this weekend moved to self-hosting the docs on digital ocean (still behind cloudflare as a CDN). We will need to add a web-hook to notify that system when new docs are pushed.

attn @QuLogic

@martinRenou
Copy link
Member

It would be great to add a section on how to make it work on colab:

from google.colab import output
output.enable_custom_widget_manager()

docs/installing.md Outdated Show resolved Hide resolved
@ianhi
Copy link
Collaborator Author

ianhi commented Mar 15, 2022

I think be best path to hosting is to use something like https://github.com/matplotlib/mpl-gui/blob/main/.github/workflows/docs.yml to put the built docs on github.

I've added this. I don't suppose there's an easy way to see them rendered on a PR?

We will need to add a web-hook to notify that system when new docs are pushed.

@QuLogic any chance of this (if not already done?) I don't think there's much left to do here

@ianhi
Copy link
Collaborator Author

ianhi commented Mar 15, 2022

Ok. I think I'm happy with this now. Can someone give a read over to review?

@QuLogic
Copy link
Member

QuLogic commented Mar 15, 2022

We will need to add a web-hook to notify that system when new docs are pushed.

@QuLogic any chance of this (if not already done?) I don't think there's much left to do here

It will only make sense to do this after you have a branch with built docs in it.

@ianhi
Copy link
Collaborator Author

ianhi commented Mar 26, 2022

Ok. I think I'm happy with this now. Can someone give a read over to review?

Absent any objections I'm going to merge this. we can always iterate more later

@ianhi ianhi merged commit 44a4ff2 into matplotlib:main Mar 26, 2022
@ianhi ianhi deleted the docs branch March 26, 2022 03:25
@ianhi
Copy link
Collaborator Author

ianhi commented Mar 26, 2022

@QuLogic we now have a gh-pages branch that contains built docs

@ianhi ianhi mentioned this pull request Mar 26, 2022
@ianhi
Copy link
Collaborator Author

ianhi commented Mar 31, 2022

@QuLogic bump - this will eventually end up at https://matplotlib.org/ipympl correct?

@tacaswell
Copy link
Member

matplotlib/matplotlib.org#15 is step 1

@tacaswell
Copy link
Member

I have the power to add the webhook, but I am unsure what the secret should be.

@QuLogic
Copy link
Member

QuLogic commented Apr 4, 2022

The secret is just a random set of characters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants