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

Add Litecoin support #91

Merged
merged 6 commits into from
May 14, 2014
Merged

Add Litecoin support #91

merged 6 commits into from
May 14, 2014

Conversation

coblee
Copy link
Contributor

@coblee coblee commented May 11, 2014

No description provided.

…6 hash of a block.

Add scrypt PoW bits validation for Litecoin blocks.
Fix to be able to start node with "-n litecoin"
@mhanne
Copy link
Contributor

mhanne commented May 12, 2014

Very nice, thanks! It's downloading the blockchain, but feels a little slow, taking ~0.6 seconds per block for the very first (empty) blocks already.. Is that to be expected because of the scrypt hashing?

@mhanne
Copy link
Contributor

mhanne commented May 12, 2014

Hm, nevermind... The performance issue seems to be specific to this one machine. The webbtc server is eating up the chain at around 0.01s/block :)

@coblee
Copy link
Contributor Author

coblee commented May 12, 2014

lian told me that his Scrypt implementation is not very optimized. So yeah, block downloading will be slow until it is optimized. The good thing is that Scrypt is only used for block PoW, so it only affects blocks. And it's most noticeable during initial block downloads.

I'm surprised to hear that your webbtc server is that fast doing scrypt hashing. Does this mean, webbtc.com will show the Litecoin blockchain soon?

@coblee
Copy link
Contributor Author

coblee commented May 13, 2014

Will now use scrypt gem if available. It's much faster.
Just "gem install scrypt"

@lian lian merged commit 4d57fe1 into lian:master May 14, 2014
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.

None yet

3 participants