Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change in PrintUtil.java and build.gradle #23

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Change in PrintUtil.java and build.gradle #23

wants to merge 5 commits into from

Conversation

h82lin
Copy link

@h82lin h82lin commented Jul 9, 2017

Change PrintUtil.java to output a json statistical file after running inference instead of a txt file. Also, the dependency to the com.google.gson library is added in build.gradle. @wmdietl

@wmdietl
Copy link
Member

wmdietl commented Jan 29, 2018

This seems to replace #20
@CharlesZ-Chen Can you see whether either one contains things we want to move to CFI?

@CharlesZ-Chen
Copy link
Member

CharlesZ-Chen commented Jan 29, 2018

This PR standardize the plain format used before by JSON format. So that we have statistic data in JSON format, which are easier to be proceed if we add statistic tool in the future. I think this might be useful for statistic benchmark. I will have a look and try to clean and migrate this to our CFI.

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.

3 participants