COIN-OR::LEMON - Graph Library

Opened 9 years ago

Closed 7 years ago

#411 closed enhancement (done)

Better image for stronglyConnectedComponents()

Reported by: Peter Kovacs Owned by: Balazs Dezso
Priority: major Milestone: LEMON 1.3 release
Component: documentation Version: hg main
Keywords: Cc:
Revision id:

Description

In the current image of stronglyConnectedComponents(), the arc arrows are so small that they can hardly recognize. This image is almost useless in this state, so the arrows must be made much larger.

Anyway, all images could be made slightly larger in the Connectivity doc module.

Attachments (2)

411-4e8787627db3-improve-images.patch (49.6 KB) - added by Peter Kovacs 7 years ago.
411-387483bf0a56-improve-image-sizes.patch (5.6 KB) - added by Peter Kovacs 7 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 9 years ago by Peter Kovacs

Owner: changed from Peter Kovacs to Balazs Dezso

comment:2 in reply to:  description Changed 9 years ago by Alpar Juttner

Replying to kpeter:

In the current image of stronglyConnectedComponents(), the arc arrows are so small that they can hardly recognize. This image is almost useless in this state, so the arrows must be made much larger.

You are right.

Anyway, all images could be made slightly larger in the Connectivity doc module.

I disagree here. The sizes are perfect IMHO. You can always zoom in an image in your browser, but think of people having smaller resolution display!

comment:3 Changed 7 years ago by Peter Kovacs

In the current image of stronglyConnectedComponents(), the arc arrows are so small that they can hardly recognize. This image is almost useless in this state, so the arrows must be made much larger.

I think this fix should be made for 1.3. As far as I see now, this image is totally useless and hence confusing. It is even better to remove it if we have no time to fix.

Anyway, all images could be made slightly larger in the Connectivity doc module.

I disagree here. The sizes are perfect IMHO. You can always zoom in an image in your browser, but think of people having smaller resolution display!

As far as I see, all images on the "Connectivity" page are about 200px wide. In fact, even I suppose a really small resolution (e.g. 800px wide), larger images (e.g. 400px wide) would just be feasible. I still suggest to increase their size at least on this page, as these images are detailed: (strongly) connected components, bi-node/edge connected components. In contrast, the images on the "Matching Algorithms" and "Traveling Salesman Problem" pages are also such small, but it does not seem problematic, as they do not contain fine details such as colored nodes and edges/arcs, short arcs/edges, directed arcs, etc.

All in all, I suggest to uniformly increase the size of all images in the doc, e.g. to 400px.

Changed 7 years ago by Peter Kovacs

Changed 7 years ago by Peter Kovacs

comment:4 Changed 7 years ago by Peter Kovacs

I attached two changesets:

  • [4e8787627db3] Improve images of the Connectivity section. All images were re-created with the old demo code of LEMON 0.x but with slightly different EPS export options. The arrows are now bigger, the arcs/edges are thicker, and distance between parallel edges is also larger. I think, these new images look better, both in their current size and in larger size, as well.
  • [387483bf0a56] Adjust image sizes in the documentation. Instead of uniform increase, I tried to achive a good size for each image. Some of them were increased (18 --> 20/24/32), while others are decreased (18 --> 8 for the node shape images). The images on the Connectivity page are now 360px wide instead of 200px.

The two patches are independent.

comment:5 Changed 7 years ago by Alpar Juttner

Resolution: done
Status: newclosed

[4e8787627db3] and [387483bf0a56] are in the main branch now.

Note: See TracTickets for help on using tickets.