-
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 data: recent blocks "Beta" computation from prior state transition Beefy Commitment / C / r #101
Comments
In particular, I believe there is no actual hashing of anything for this 32-byte output so we could put a 4 byte service id and the rest can be all 28 0s -- is this a good idea?
Based on C.9 is it We can provide an exact derivation of the the mmr with agreement on the above new concepts.
but if we added a
where |
I'm not sure if I'm following correctly, but I would think if it was intended to be null then register 7 and 8 wouldn't point to a valid memory location. At the end of the program i have reg7=4278058952 and reg8=32 and thats the memory location where I'm pulling this value from: 0x549611b000000000000000000000000000000000000000000000000000000000 |
We put (If not, what would?) We'll generate a derivation with the answer being "yes, it does trigger the condition" since we have no answer to the above. The Same question posed in GP Chat ![]() |
Addresses #101 in that $leaf had to be removed from our WBT implementation to match Added beefy commitment / accumulation root / recent blocks logging
I'm not aware of a requirement that an output is 32 characters. The condition here is basically as long as the program halts and the memory specified is in a readable location. ![]() |
Ok we found an issue on our side (a This now correctly computed root is used to compute the Beta/RecentBlocks of the NEXT block here That is, the This
Hmm, hmm, a clarification will be useful ! |
I use it for the same block. My STF just has two steps for beta. Where did you see that it's for the next block? |
The "present block" reference here we take to be describing Eq 4.17 We think the use of C in both Eq 4.7 AND Eq 4.17 is confusing, and something should be done about it notationally. |
I'm aligned with this interpretation; I don't think Gavin means two different things with the same letter. My internal process is:
|
Since the C of 4.7 is in fact the SAME as the C of 4.17 (this is definitely not reader-friendly) and confirmed to not need some different notation, we'll consider this case closed, will republish a corrected dataset. This means we also don't need a "accumulationRoot" feature in StateTransition. |
solved in 0.6.2.7 |
Our delta (services) objects finally fully match after accumulation!
I now have a single difference in post-state beta after running assurances/5.
Yours:
BlockInfo{hash='0xbea2a0bd18d90f052853f81a9747431b0b44f65f5f16eb2ce116dbef5ba2fea5', mmr=Mmr{peaks=[null, 0xad3228b676f7d3cd4284a5443f17f1962b36e491b30a40b2405849e597ba5fb5, 0xb4c11951957c6f8f642c4af61cd6b24640fec6dc7fc607ee8206a99e92410d30]}, stateRoot='0x0000000000000000000000000000000000000000000000000000000000000000', reported=[]}]
Mine:
BlockInfo{hash='0xbea2a0bd18d90f052853f81a9747431b0b44f65f5f16eb2ce116dbef5ba2fea5', mmr=Mmr{peaks=[null, 0x755624dca12d3dead2d2e064a4e060ba5cf6d5d9280acdc4c552cccdbde3c9cf, 0xb4c11951957c6f8f642c4af61cd6b24640fec6dc7fc607ee8206a99e92410d30]}, stateRoot='0x0000000000000000000000000000000000000000000000000000000000000000', reported=[]}]
My service hash pairs
service 0 - hash: 0x549611b000000000000000000000000000000000000000000000000000000000
After encoding (E4(s)⌢ E(h)): 0x00000000549611b000000000000000000000000000000000000000000000000000000000
Accumulate root: - MB([0x00000000549611b000000000000000000000000000000000000000000000000000000000]) = 0x253e7b79a6f60c38d85fa99ae8f0fbf3072e5bb869d39e4319ef609b0395e14b
The text was updated successfully, but these errors were encountered: