Marble diagram for Observable.concatMapMaybe is wrong

[boris-petrov]

https://reactivex.io/RxJava/3.x/javadoc/io/reactivex/rxjava3/core/Observable.html#concatMapMaybe-io.reactivex.rxjava3.functions.Function-

The diagram says concatMap instead of concatMapMaybe and also shows two items being emitted by the mapping function which is impossible in a Maybe. I guess the diagram was copied from concatMap. :)

[akarnokd]

Yes, we have still a lot of outdated and copy-paste diagrams.

No promises when they will be updated.

[dhrax21]

Hey can I work on this issue

[akarnokd]

Sure.

[dhrax21]

but what diagram should i replace with?

[akarnokd]

You have to draw it yourself, then post the png here.

If you have offline Excel, you could do it in our style:

• Maybe marbles
• Single marbles
• Completable marbles
• Mixed use marbles

Or use Draw.IO but it's a bit fiddly for this type of diagram work.

[dhrax21]

Sorry I don't wanna work on this 👍

[CHIRANTH-24]

Hello @boris-petrov , I was able to understand the first part. Wanted help for the second.

[boris-petrov]

@CHIRANTH-24 - note that it's Maybe, not MayBe.

As for the second part - the diagram maps a single source item (the red circle) to multiple results (the white and red squares). It should be only a single white and a single red square. Same for the green and blue ones. Check the diagram for flatMapMaybe.