debux icon indicating copy to clipboard operation
debux copied to clipboard

Feature request: optionally return results instead of printing

Open Cyrik opened this issue 3 years ago • 8 comments

I've been thinking that it would be great to be able to get the debug results so that it is possible to show them in different UI's, similar to re-frame-debux, but without the re-frame dependency. Other use cases would be logging or piping into REBL, reveal or portal. I've already started a prototype to play around with designs for the interface. If you are interested in the feature I'll fork and push my changes.

Cyrik avatar Sep 14 '21 20:09 Cyrik

I have been interested in this feature as well. Please send me your pull request for this feature and I will integrate it as much as possible.

Thanks for this feature request and upcoming your pull request!!

philoskim avatar Sep 15 '21 02:09 philoskim

i've been playing around with a representation of the results that has all the data in a way that can be used to display it. I'm not happy yet, but your input is very welcome

Cyrik avatar Sep 20 '21 15:09 Cyrik

Your pull request was moved to a new branch 'multiple-outputs'.

philoskim avatar Sep 23 '21 03:09 philoskim

@philoskim hey, sorry I didn't get back to this for some time. Hoping to continue working on this sometime this week and try to make debux a dependency of my own lib.

Cyrik avatar Nov 30 '21 09:11 Cyrik

I'm also quite interested in this, as I want to display the trace data-structure in portal. I know that some of this has been done, because I see that @Cyrik 's fork of debux can be used as the inner layer of omni-trace. How much of that work is upstreamable? If it's not, was there some agreement of the shape of the datastructure?

meditans avatar May 06 '22 17:05 meditans

Sorry it took so long @meditans. My version should be mergable. I think there's a tiny bit of cleanup left to not waste memory if you're only printing.

Cyrik avatar Jul 30 '22 13:07 Cyrik

Would this also allow the result to be used in a repl?

deejayem avatar Nov 15 '22 12:11 deejayem

Yes it would, although as @meditans said, my implementation is still a bit messy.

Cyrik avatar Nov 15 '22 23:11 Cyrik