network-quality / goresponsiveness

A draft-ietf-ippm-responsiveness client in Go.
GNU General Public License v2.0
133 stars 10 forks source link

In `parallel_testing` make sure that result output order is consistent #65

Closed hawkinsw closed 10 months ago

hawkinsw commented 10 months ago

https://github.com/network-quality/goresponsiveness/commit/250d8272c3d91e247b9a53c5dc44651f6ee17761#commitcomment-136014531

@moeller0 made a great point that when doing parallel testing, the order of the output depends on which of the directions finishes first. This is definitely not ideal!

Will

hawkinsw commented 10 months ago

Check out: 2d058b887998252353f41019b03763886f42d832, @moeller0

hawkinsw commented 10 months ago

@moeller0, I have made further cleanups: https://github.com/network-quality/goresponsiveness/commit/555398e4e005d3b1036d92d534b906d03437c3af

hawkinsw commented 10 months ago

Thanks to the feedback from @moeller0, we now have much better looking output. Closing this after 218f2e6d235c4e947a04a39e4e09acde7675de6a!