You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Clojure Dev
Hello,
I would like to contribute to the language (I signed the contributor agreement already).
I noticed that the flatten function does not provide a transducer, I am proposing to implement one. But before I spend for effort on a PR, I would like to know if anyone is already working on that and how likely it is to be accepted.
Regards, Vincent.
Alex Miller
unread,
Feb 18, 2018, 3:58:16 PM2/18/18
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to cloju...@googlegroups.com
Hi Vincent,
Generally cat and mapcat transducers cover most cases. I think a flatten transducer is pretty low priority.
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Clojure Dev
Hi Alex and Tom,
@Alex: cat and mapcat do not cover the case of arbitrarily deep tree structures, the user still has to build his own solution for that, and when they do they often end up using a mix of transducers and sequences which is not the best for performances.
@Tom: that's a good idea. I finally decided to implement a transducer version of `tree-seq` instead of `flatten` and I submitted it to his project already: https://github.com/cgrand/xforms/issues/20