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

Add alt text to Dask docs #1

Merged
merged 13 commits into from
Dec 4, 2021
Merged

Add alt text to Dask docs #1

merged 13 commits into from
Dec 4, 2021

Conversation

scharlottej13
Copy link
Owner

@scharlottej13 scharlottej13 commented Nov 30, 2021

Welcome!

This is the working PR for the Dask Community alt text mini sprint! This is where we collaborate during the event before we send a clean contribution over to the main repo.

How to contribute

For the full info about this event, please visit the HackMD agenda.

During the event, you can contribute alt text to any of the images on currently used in a blog post. Please contribute via suggestions so we can easily review each other's work.

If you plan to work on a file, let people know by commenting below. We can have multiple suggestions for the same alt text, but it would be nice if we spread out to cover more images.

If you think an image is missing from these files, let @scharlottej13 know and she'll add it to the PR.

Images in this PR

To make collaborating easier, comment below with the number of the image you'd like to work on. You can do this as a group, and you can write alt text for as many images as you'd like.

file: docs/source/logos.rst

  • (example) line 6

file: docs/source/why.rst rendered docs

  • 1. line 19
  • 2. line 27

file docs/source/10-minutes-to-dask.rst rendered docs

  • 3. line 376
  • 4. line 394
  • 5. line 410

file docs/source/array-chunks.rst rendered docs

  • 6. line 270
  • 7. line 277
  • 8. line 299
  • 9. line 315
  • 10. line 322

file docs/source/array-design.rst rendered docs

  • 11. line 7

file docs/source/array-overlap.rst rendered docs

  • 12. line 35
  • 13. line 41
  • 14. line 48

file docs/source/delayed.rst rendered docs

  • 15. line 25
  • 16. line 113

file docs/source/develop.rst rendered docs

  • 17. line 368

file docs/source/graphs.rst rendered docs

  • 18. line 60

file docs/source/graphviz.rst rendered docs

  • 19. line 37

file docs/source/optimize.rst rendered docs

  • 20. line 62
  • 21. line 101
  • 22. line 117
  • 23. line 137
  • 24. line 154

file docs/source/order.rst rendered docs

  • 25. line 104
  • 26. line 141

file docs/source/shared.rst rendered docs

  • 27. line 91
  • 28. line 104
  • 29. line 106
  • 30. line 121

@isabela-pf
Copy link

Here's a proposed checklist for alt text style (may be helpful for review):

The alt text has

  • Correct spelling and no typos
  • Periods and commas where relevant
  • No more than three short, complete sentences
  • A logical way of fitting in the rest of the documentation
  • Consistent text and/or descriptions for the same elements in different images
  • A description of any text in the image

docs/source/why.rst Outdated Show resolved Hide resolved
docs/source/shared.rst Outdated Show resolved Hide resolved
docs/source/delayed.rst Outdated Show resolved Hide resolved
docs/source/why.rst Outdated Show resolved Hide resolved
docs/source/develop.rst Outdated Show resolved Hide resolved
docs/source/delayed.rst Outdated Show resolved Hide resolved
@@ -119,6 +119,7 @@ are good at workflows with non-trivial cross-task
communication; they have been removed from the plot.

.. image:: images/scaling-edges.png
:alt:

Choose a reason for hiding this comment

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

Suggested change
:alt:
:alt: Two graph panels. Left: Cost of entire graph, Duration versus Number of edges per task. Dask.async and dask.threaded curves both increase linearly, with the latter consistently higher by at least a factor of 3. Right: Cost per edge, with the same axes. Both curves start from a high value for small number of edges, but reach an asymptote where dask.threaded is a factor of a few higher than dask.async .

@@ -7,7 +7,7 @@ Overview
.. image:: images/array.png
:width: 40 %
:align: right
:alt: A dask array
:alt: A dask array.

Choose a reason for hiding this comment

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

I'm working on this!

docs/source/order.rst Outdated Show resolved Hide resolved
docs/source/order.rst Outdated Show resolved Hide resolved
docs/source/array-design.rst Outdated Show resolved Hide resolved
Co-authored-by: Martin Durant <martindurant@users.noreply.github.com>
Co-authored-by: Ian Rose <ian.r.rose@gmail.com>
Co-authored-by: Pavithra Eswaramoorthy <pavithraes@outlook.com>
Co-authored-by: Julia Signell <jsignell@gmail.com>
Co-authored-by: Isabela Presedo-Floyd <50221806+isabela-pf@users.noreply.github.com>
Co-authored-by: Charles Blackmon-Luca <20627856+charlesbluca@users.noreply.github.com>
docs/source/shared.rst Outdated Show resolved Hide resolved
docs/source/shared.rst Outdated Show resolved Hide resolved
@@ -391,6 +392,7 @@ triggering computation, we can inspect the task graph to figure out what's going
>>> b.visualize()

.. image:: images/10_minutes_array_graph.png
:alt: Two neighboring blocks with thin strips near their shared border representing the shared data.

Choose a reason for hiding this comment

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

Is this alt text referring to a different image? It doesn't seem like it goes with this one.

I was expecting something like "Dask task graph of ... blah blah blah" for alt text belonging to images/10_minutes_array_graph.png

Choose a reason for hiding this comment

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

Yes, this very definitely does not go with this image. I've added an alt-text suggestion for the actual image here below.

@@ -7,7 +7,7 @@ Overview
.. image:: images/array.png
:width: 40 %
:align: right
:alt: A dask array
:alt: 12 rectangular blocks arranged as a 4-row, 3-column layout. Each block includes 'x' and its location in the table starting with ('x',0,0) in the top-left, and a size of 5x8.

Choose a reason for hiding this comment

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

👍

@@ -34,20 +34,20 @@ Consider two neighboring blocks in a Dask array:

.. image:: images/unoverlapping-neighbors.png
:width: 30%
:alt: un-overlapping neighbors
:alt: Two neighboring blocks which do not overlap.

Choose a reason for hiding this comment

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

👍


We extend each block by trading thin nearby slices between arrays:

.. image:: images/overlapping-neighbors.png
:width: 30%
:alt: overlapping neighbors
:alt: Two neighboring block with thin strips along their shared border representing data shared between them.

Choose a reason for hiding this comment

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

👍


We do this in all directions, including also diagonal interactions with the
overlap function:

.. image:: images/overlapping-blocks.png
:width: 40%
:alt: overlapping blocks
:alt: A two-dimensional grid of blocks where each one has thin strips around their borders representing data shared from their neighbors. They include small corner bits for data shared from diagonal neighbors as well.

Choose a reason for hiding this comment

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

👍

@@ -23,7 +23,7 @@ directly with a light annotation of normal python code:
>>> z.visualize()

.. image:: images/inc-add.svg
:alt: simple task graph created with dask.delayed
:alt: A task graph with two "inc" functions combined using an "add" function resulting in an output node.

Choose a reason for hiding this comment

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

👍

@@ -112,7 +112,7 @@ example, because these functions are already very small and fast.)

.. image:: images/delayed-inc-double-add.svg
:align: right
:alt: simple task graph created with dask.delayed
:alt: A task graph with many nodes for "inc" and "double" that combine with "add" nodes. The output of the "add" nodes finally aggregate with a "sum" node.

Choose a reason for hiding this comment

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

👍

@@ -102,6 +102,7 @@ to get a true representation of what order the tasks will be executed in.


.. image:: images/order-failure.png
:alt: Complex task graph of several vertical node chains at the output, and a few input sub-trees. In between these sections, there is a many-to-many area of crossing dependency arrows. The color coding of the output trees is interleaved without a clear progression.

Choose a reason for hiding this comment

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

👍

@@ -139,6 +140,7 @@ see the effect of ordering:


.. image:: images/order-success.png
:alt: Complex task graph of several vertical node chains at the output, and a similar number of input blocks. The outputs and inputs are linked by simple nodes of a few inputs each, laid out without significant crossover between sections of the tree. The color coding of the output chains shows clear progression in the order of execution with each output color having a corresponding input of the same color.

Choose a reason for hiding this comment

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

👍

scharlottej13 and others added 2 commits December 3, 2021 09:55
thanks `@GenevieveBuckley` for all the suggestions!

Co-authored-by: Genevieve Buckley <30920819+GenevieveBuckley@users.noreply.github.com>
Co-authored-by: Genevieve Buckley <30920819+GenevieveBuckley@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
8 participants