-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: Add figure for navigating help resources #9147
Conversation
Very cool! Thanks @JessicaS11 Couple of suggested changes (maybe others will disagree, I see a draft of this was reviewed in the discussion):
|
Readthedocs is failing with: |
looks like that was just a temporary hiccup: https://readthedocs.org/projects/xray/builds/24763397/ |
Thanks for sharing these - I'll commit once I've addressed them all.
I'll update these
Good call!
Done!
I guess I was thinking the case where the issue is a proposal that needs "approval" or feedback for the contributor to work on and get to the point of submitting a PR. Bug report issues aren't generally going to be "quiet" because maintainers are likely to be working on fixing them.
What is a "gentle ping" then? Simply posting a comment in the PR? |
Sorry, yes, I'm not sure the best wording, I was thinking
IME it's quite rare that things need approval to get started, at least in projects like xarray. And there are almost 1000 issues so pinging for updates is somewhat disruptive. So I would probably:
Is that OK? I don't mean to be wordsmithing. (It's actually quite interesting to have to make legible lots of implicit understanding of how to operate.) |
It's great - I completely agree on the making explicit the implicit being an interesting exercise. I went with "Comment on your PR" - to me a "ping" means an @ mention or Slack/email message (basically explicitly contacting a group or individuals), which I'm sure is one of many definitions people have for it! |
I think this is looking really good! Thanks @JessicaS11 ! |
Thanks! I'm going to see if I can get it to render properly in the docs before merging (hence ongoing changes). Right now it's building successfully but showing up as text... |
Hey team - I'm hitting a wall here on how to get this rendered. Specifically, it appears the sphinxcontrib.mermaid extension uses an older version of mermaid (10.2.0) than the mermaid.live editor I used to create the flowchart (10.9.1). As a result, there are syntax errors. My understanding of the solution pathways are:
Any help or insight is greatly appreciated! |
I found a setting to set a custom mermaid version: https://sphinxcontrib-mermaid-demo.readthedocs.io/en/latest/#config-values |
@dcherian Thank you!! |
* main: promote floating-point numeric datetimes to 64-bit before decoding (pydata#9182) also pin `numpy` in the all-but-dask CI (pydata#9184) temporarily remove `pydap` from CI (pydata#9183) temporarily pin `numpy<2` (pydata#9181) Change np.core.defchararray to np.char (pydata#9165) (pydata#9166) Fix example code formatting for CachingFileManager (pydata#9178) Slightly improve DataTree repr (pydata#9064) switch to unit `"D"` (pydata#9170) Docs: Add page with figure for navigating help resources (pydata#9147) Add test for pydata#9155 (pydata#9161) Remove mypy exclusions for a couple more libraries (pydata#9160) Include numbagg in type checks (pydata#9159) Improve zarr chunks docs (pydata#9140)
whats-new.rst