Adding mapValuesF to MapOps

[falconepl]

Adding mapValuesF to MapOps

[benhutchison]

My first reaction is that we should focus on education rather than creating extra methods that are aliases. I remember the first time I used parUnorderedTraverse it took me ages to figure out that what was I needed. But having learnt it, it becomes another useful tool in the kit.

…

On Sun., 24 Apr. 2022, 7:51 pm Przemek Sokół, ***@***.***> wrote: Do you think it would be useful to have mapValuesF in MapOps? I recently had to get from Map[A, B] to F[Map[A, C]] (instead of Map[A, F[C]]) when mapping values with B => F[C] function. After some time I've realized that it can be done with parUnorderedTraverse, but it wasn't too obvious - my first thought was to check if there's mapValuesF in Mouse. When it comes to mapValuesF definition for MapOps, it could look like this: import cats.syntax.parallel._ def mapValuesF[F[_], C](f: B => F[C])(implicit F: CommutativeApplicative[F], P: Parallel.Aux[F, F]): F[Map[A, C]] = map.parUnorderedTraverse(f) What do you think? There's is a quite popular thread <https://stackoverflow.com/questions/53945996/cats-effecthow-to-transform-mapx-ioy-to-iomapx-y> on StackOverflow related to this topic so maybe it would be useful to have mapValuesF here — Reply to this email directly, view it on GitHub <#335>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAXJZCKWKLBUT53SAZCJG3VGUKR7ANCNFSM5UGAJJVA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[falconepl]

Good point. I haven't noticed that Mouse tries to avoid aliases for existing methods - and surely, mapValuesF in this case would be nothing more than an alias (maybe a bit easier to find than parUnorderedTraverse but that's all). Also, that "focus on education" you've mentioned makes a lot of sense here. Thanks for the feedback! I'm closing the issue then 🙂

[danicheg]

I altogether agree with Ben. My point is that mouse should bring extra functionality regarding the cats, not the alternatives to existing approaches.