update: updating sequence diagrams #4646
Conversation
There was a problem hiding this comment.
Should it be .filter, .map, and .take (with a period .) when mentioning these functions in this and the rest of the images?
There was a problem hiding this comment.
as discussed - let's keep it without . as these are not extension functions
There was a problem hiding this comment.
Maybe a small detail, but I think the vertical line separating the not-processed words is a bit crooked. You can see the distance between the line and the left-hand rectangles is different.
There was a problem hiding this comment.
excellent idea - updated it
docs/topics/sequences.md
Outdated
|
|
||
| In this example, the sequence processing takes 18 steps instead of 23 steps for doing the same with lists. | ||
| In this example, by processing elements lazily and stopping once four items are found, sequences reduce the total number |
There was a problem hiding this comment.
Would something simpler work? Like:
In this example, lazy processing of elements and stopping after finding four items minimizes the number of operations compared to a list approach.
There was a problem hiding this comment.
Hmm - I like this approach but I prefer reduce over minimize - it's not necessarily minimized just reduced - but could probably be further reduced with some other optimization step 🤔
Creating new diagrams to demonstrate how using sequences differ from using a list approach.
Related issue: KT-72898