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

Output order #650

Open
Leonardiae opened this issue Jan 20, 2022 · 2 comments
Open

Output order #650

Leonardiae opened this issue Jan 20, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@Leonardiae
Copy link
Collaborator

BTCSQ blocks have the PoW reward as the last output

It looks as if third parties (ChainZ / Miningpoolstats) use the first output to recognise who / which pool found that block. But in the case of BTCSQ this results in showing the address of a random PoP miner (the one who first appears in the list sorted by address it seems)

Not sure if there is an official convention but for other blockchains usually blocks have the PoW reward as the first output.

I guess in theory every miner can order the outputs added to a block in the order as he likes but assuming almost everyone will go with the default functionality it would be great if by default the output order has the PoW reward as the first output.

@Warchant Warchant moved this to Backlog in Proof-of-Proof Jan 20, 2022
@Warchant Warchant moved this from Backlog to Triaged in Proof-of-Proof Jan 20, 2022
@Warchant Warchant added the enhancement New feature or request label Jan 20, 2022
@Warchant Warchant self-assigned this Jan 20, 2022
@Warchant
Copy link
Collaborator

Output order is not strictly defined. Our pool adds POW payout as last payout, whereas generatetoaddress in BTCSQd as first. If we limit to either variant, then it means a hard fork in BTCSQ... Not sure it is worth it.

@Leonardiae
Copy link
Collaborator Author

Maybe we can save this and implement if we have a hardfork in the future?
Can we change it already for altchains who implement PoP?

@Warchant Warchant moved this from Triaged to Backlog in Proof-of-Proof Jan 31, 2022
@Warchant Warchant added this to the BTCSQ Hardfork milestone Jan 31, 2022
@Warchant Warchant removed their assignment Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

2 participants