Skip to content
This repository has been archived by the owner on Feb 1, 2019. It is now read-only.

perf-metrics.md: Add performance metrics #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ksdme
Copy link
Member

@ksdme ksdme commented Jun 12, 2018

Add performance metrics data from Phase-1 to the repo.

Copy link
Member

@gaocegege gaocegege left a comment

Choose a reason for hiding this comment

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

I think it is still WIP, and we need to add sections about the implementation in coala-vs-code

@ksdme ksdme force-pushed the metrics branch 3 times, most recently from e0fcb85 to 8046aef Compare June 12, 2018 05:50
@ksdme
Copy link
Member Author

ksdme commented Jun 12, 2018

@gaocegege PTAL, do we need to have sections instead of tabular column?

@gaocegege
Copy link
Member

Oh, this works for me. Could we add some figures to illustrate the performance?

@gaocegege
Copy link
Member

Maybe add a dimension to

legacy, max = 1, max = 2, max = 3

@ksdme
Copy link
Member Author

ksdme commented Jun 12, 2018

On it.

Add performance metrics data from Phase-1
to the repo.

Closes coala#10
Copy link
Member

@gaocegege gaocegege left a comment

Choose a reason for hiding this comment

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

ack

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

2 participants