Skip to content

CP for #300: Enable recall/latency charts on results#26

Merged
filipecosta90 merged 2 commits intomulticlient_toolfrom
multiclient_latencies
Jun 9, 2022
Merged

CP for #300: Enable recall/latency charts on results#26
filipecosta90 merged 2 commits intomulticlient_toolfrom
multiclient_latencies

Conversation

@filipecosta90
Copy link
Copy Markdown

This PR cherry-picks the proposed changes in erikbern#300 into multiclient_tool

Copy link
Copy Markdown

@GuyAv46 GuyAv46 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. One fix

@filipecosta90 filipecosta90 requested a review from GuyAv46 June 9, 2022 22:22
Copy link
Copy Markdown

@GuyAv46 GuyAv46 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@filipecosta90 filipecosta90 merged commit 46c5430 into multiclient_tool Jun 9, 2022
@filipecosta90 filipecosta90 deleted the multiclient_latencies branch June 9, 2022 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants