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 x16rt algo #338

Open
wants to merge 11 commits into
base: next
Choose a base branch
from
Open

Add x16rt algo #338

wants to merge 11 commits into from

Conversation

SKYPOOLdotCO
Copy link

This x16rt currently works with the upcoming Gincoin fork, which uses standard block header - not for Veil.

@tpruvot
Copy link
Owner

tpruvot commented Feb 3, 2019

sph_sha256_context is already in sha3/sph_sha2.c


uint32_t *in32 = (uint32_t*) input;
uint32_t ntime = in32[17];
uint8_t timeHash[8]; /* was uint32_t _ALIGN(64) timeHash[8]; */
Copy link
Owner

@tpruvot tpruvot Feb 3, 2019

Choose a reason for hiding this comment

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

this is incorrect, a sha256d is 32 bytes (256bits)

Copy link
Author

@SKYPOOLdotCO SKYPOOLdotCO Feb 3, 2019

Choose a reason for hiding this comment

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

I'll try to correct this. Also replace sha256d with existing sha256_double_hash.

@olalawal
Copy link

So when will veil be added ?

@serhii-cherkez
Copy link

Does not work!

@gpuminingir
Copy link

does this work now? has it been tested with veil or gin?

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.

6 participants