Skip to content

Conversation

@hugo-dc
Copy link
Member

@hugo-dc hugo-dc commented Jul 31, 2020

No description provided.

@axic
Copy link
Member

axic commented Jul 31, 2020

eosvm doesn't seem to show up in the charts?

@hugo-dc
Copy link
Member Author

hugo-dc commented Aug 3, 2020

I have rebased master, and included eos-vm in the notebook so the charts get generated.

@hugo-dc hugo-dc requested a review from axic August 3, 2020 01:41
@axic
Copy link
Member

axic commented Aug 6, 2020

Can you add eosvm to the README?

Copy link
Contributor

@jwasinger jwasinger left a comment

Choose a reason for hiding this comment

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

The notebook is going to be removed from the repo: #152 . Can you move notebook changes in this PR to the new script for generating charts: https://github.com/ewasm/benchmarking/blob/master/notebooks/wasm-engines.py ?

@hugo-dc
Copy link
Member Author

hugo-dc commented Aug 12, 2020

I already modified the script https://github.com/ewasm/benchmarking/pull/150/files#diff-a6dee7477faf26016717d019cdc8bf1b , althought I modify it manually, do you want me to use the nbconvert tool instead?

@hugo-dc
Copy link
Member Author

hugo-dc commented Aug 12, 2020

Now I converted the notebook to a script, and included the charts generated by the script.

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