Skip to content

Add x16rt algo#338

Open
SKYPOOLdotCO wants to merge 11 commits into
tpruvot:nextfrom
SKYPOOLdotCO:next
Open

Add x16rt algo#338
SKYPOOLdotCO wants to merge 11 commits into
tpruvot:nextfrom
SKYPOOLdotCO:next

Conversation

@SKYPOOLdotCO

Copy link
Copy Markdown

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

@tpruvot

tpruvot commented Feb 3, 2019

Copy link
Copy Markdown
Owner

sph_sha256_context is already in sha3/sph_sha2.c

Comment thread stratum/algos/x16rt.c

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

@tpruvot tpruvot Feb 3, 2019

Copy link
Copy Markdown
Owner

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)

@SKYPOOLdotCO SKYPOOLdotCO Feb 3, 2019

Copy link
Copy Markdown
Author

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
Copy Markdown

So when will veil be added ?

@serhii-cherkez

Copy link
Copy Markdown

Does not work!

@gpuminingir

Copy link
Copy Markdown

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