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

["Request"] Use FlowMarbles in Arrow Fx Coroutines Flow operators documentation #2458

Open
nomisRev opened this issue Jul 28, 2021 · 5 comments

Comments

@nomisRev
Copy link
Member

What would you like to see?

Marble diagrams for Flow extensions.

https://github.com/rougsig/flowmarbles

@rougsig
Copy link

rougsig commented Aug 18, 2021

If needed, i can add the iframe flag for include diagram only, without extra layout.
Also i can add extra operators from arrow to diagram, if arrow-kt flow operators can support kotlin/js
Smt like that https://flowmarbles.com/#flatMapLatest?iframe=true

@nomisRev
Copy link
Member Author

nomisRev commented Aug 19, 2021

Hey @rougsig,

If needed, i can add the iframe flag for include diagram only, without extra layout.
Also i can add extra operators from arrow to diagram, if arrow-kt flow operators can support kotlin/js

That'd be awesome! 😍 I have no idea how to otherwise go about including this into our Jekyll website 😅
Next month Arrow 1.0.0 will release with JS support, then I'd be happy to contribute it to your repo to add Arrow examples.

@rougsig
Copy link

rougsig commented Aug 20, 2021

sure no problems

@rougsig
Copy link

rougsig commented Oct 28, 2021

Hey @nomisRev

Can you tell me which operators you like to see on marble?

@nomisRev
Copy link
Member Author

Hey @rougsig,

My apologies, for the late response. All the extensions on Flow can be found here: https://github.com/arrow-kt/arrow/blob/main/arrow-libs/fx/arrow-fx-coroutines/src/commonMain/kotlin/arrow/fx/coroutines/flow.kt

I think these are the most interessting ones to show:

  • parMap
  • parMapUnordered
  • metered/fixedRate

The retry extension based on Schedule is going to be hard to show unless we fix it to a specific input.
And mapIndexed is probably not worth adding :D

If there is anything I can help with feel free to reach out to me on Kotlin Slack too.

This would be so cool to have. Thanks for your interest to help 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants