DOC: Include np.long in arrays.scalars.rst
#25030
Conversation
There was a problem hiding this comment.
I think there's a desire to keep the .dia file and only modify that file and regenerate the png and PDF, from inside Dia see #23252 (comment). I'm able to open the file locally if I do sudo apt install dia then dia path/to/file.dia.
If that ends up being too cumbersome we could come back to how to handle the diagram files the numpy docs have in a few places but please give it a try first.
|
@ngoldbaum Sure! I restored&updated the Nevertheless, I would argue that all diagrams could be migrated to e.g. Google Drawings or draw.io (both free), as the Dia is ancient and rather cumbersome to use. (I hope there isn't another reason to stick to it) |
Sure, I think work happened along these lines in #18501, but the diagrams never got updated . Ideally (I think) the source for the diagram would live in the docs. I'd argue it makes the most sense to use the built-in support in sphinx for the dot format, see e.g. #18501 (comment).
I'm pretty sure the only reason is inertia. |
|
Let's just go ahead and merge this, we can fix doc issues in a followup. |
Addresses #25015.
Hi @ngoldbaum, in this PR I updated
arrays.scalars.rstso it includesnp.longand updated diagrams. (the other one-line changes are fixes for some warnings that I found when building locally)