-
Notifications
You must be signed in to change notification settings - Fork 10
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
Assurances/2_000 - accumulation arguments/operand #107
Comments
In our case, there are two services that are going to accumulate. Service 1: Accumulation arguments:
Length: 88 Service ID: 552088286 Operands[0]: Work result: OK
work result payload hash:
work package hash:
authorizer output:
Servcie 2: Accumulation arguments:
Length: 108 Service ID: 272334947 Operands[0]: Work result: OK
work result payload hash:
work package hash:
authorizer output:
|
Hey @clw8998, where did service # 552088286 and 272334947 come from? The pre-state and post-state for this transition has 3 services, 0, 1065941251 and 2953942612. I am also wondering about the work package hash in both accumulations. I believe this should be 0x9cb43cfee9fe3afca97522dfe5249e2119da6da0ef686e7726b4a64ef5df4de6 ![]() |
Oops! It seems like I was on the wrong branch. The two service indices should be 1065941251 and 2953942612, and their accumulation argument details are as follows: Service 1: Accumulation arguments:
Length: 88 Service ID: 1065941251 Operands[0]: Work result: OK
work result payload hash:
work package hash:
authorizer output:
Servcie 2: Accumulation arguments:
Length: 108 Service ID: 2953942612 Operands[0]: Work result: OK
work result payload hash:
work package hash:
authorizer output:
|
@clw8998 which one goes through accumulation first in your implementation? |
GP doesn't care which service goes first, the "single accumulate" We happen to use maps that also have no specific ordering during iteration so either of these could be done first but it should not affect the result. If one of these services had a prereq (as we do have in the |
@sourabhniyogi the order may matter when it comes to the commitment map C.. although maybe I missed in the GP if that gets sorted or not |
posted new 0.6.2.8 - kindly check @jaymansfield |
We have a difference in our accumulation arguments for the first pvm invocation for assurances/2_000.
My implementations has 108 bytes while yours seems to be 88 bytes.
Mine:
0x18000000549611b00100208c30f2c101674af1da31769e96ce72e81a4a44c89526d7d3ff0a1a511d5f3c9f0e5751c026e543b2e8ab2eb06099daa1d1e5df47778f7787faab45cdf12fe3a82a929d319baf939ea37f7cfd488d736eecca8ce04ba8098dda4e829b928d0b1d00
tau = 24
service ID = 2953942612
operands[0] =
work result: OK - 0x8c30f2c101674af1da31769e96ce72e81a4a44c89526d7d3ff0a1a511d5f3c9f
work result payload hash: 0x0e5751c026e543b2e8ab2eb06099daa1d1e5df47778f7787faab45cdf12fe3a8
work package hash: 0x9cb43cfee9fe3afca97522dfe5249e2119da6da0ef686e7726b4a64ef5df4de6
authorizer output: 0x
The text was updated successfully, but these errors were encountered: