Skip to content
This repository has been archived by the owner on Mar 18, 2022. It is now read-only.

digest64 #51

Merged
merged 2 commits into from
May 4, 2021
Merged

digest64 #51

merged 2 commits into from
May 4, 2021

Conversation

g11tech
Copy link
Contributor

@g11tech g11tech commented May 1, 2021

#49

PS: this work has not been funded or granted by anyone. if you think this adds value to the ecosystem you could tip the author at: 0xb44ddd63d0e27bb3cd8046478b20c0465058ff04

@CLAassistant
Copy link

CLAassistant commented May 1, 2021

CLA assistant check
All committers have signed the CLA.

assembly/index.ts Outdated Show resolved Hide resolved
Copy link
Member

@wemeetagain wemeetagain left a comment

Choose a reason for hiding this comment

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

Awesome job @g11in

Solid 5% speedup (on my local machine)

@g11tech
Copy link
Contributor Author

g11tech commented May 4, 2021

Awesome job @g11in

Solid 5% speedup (on my local machine)

thanks :) but just 5% ? my local benchmarking gives 20% consistently!

Copy link
Contributor

@dapplion dapplion left a comment

Choose a reason for hiding this comment

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

🎉 Looks great! @wemeetagain is the WASM blob under the size limit to be loaded syncronously?

@wemeetagain
Copy link
Member

thanks :) but just 5% ? my local benchmarking gives 20% consistently!

Yea, lower than I expected, perhaps its just on my hardware

is the WASM blob under the size limit to be loaded syncronously?

Yes, looks like the optimized wasm blob is 2.6kb, under the 4kb threshold

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

Successfully merging this pull request may close these issues.

4 participants