-
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
Incorrect items/bytes for service #82
Comments
Pre-stateFor service 0 the prestate of 1_004
is the same as genesis so I think we have to agree on:
Derivation is from GP 9.3 ![]()
Which part do you derive a different answer? Post-state[After we get on the same page on the above ...] You are correct -- our bootstrap has a
and then similarly for the new service
like
that would be most helpful so we can achieve consensus on the correct answer most quickly! |
My mistake on the pre-state. The values are correct. For service 0 I am getting bytes=359, items=3. As for the new service, until we get the same accumulation results it's hard to say what the correct output should be for it |
Great we agree on bytes=359, items=3. Have to do some macro data refinement... |
Addresses * fix of serviceid generation + Demonstrate Option<Hash> in assurances/orderedaccumulation data #74 as reported by @davxy - there was an extra 0x20 which should not have been there since 32-byte hashes are special, see #74 for derivation, confirmed by @jaymansfield + @davxy - added SetServiceStorage lines in { assurances.txt, orderedaccumulation.txt } * fix of Incorrect items/bytes for service #82 as reported by @jaymansfield - added a_i + a_o lines in { assurances.txt, orderedaccumulation.txt } for derivation * fix of state merklization for storage as reported by @dakk - previous was 0.4.x based (TODO: check tool https://github.com/dakk/jambiato0
…#87) Addresses * fix of serviceid generation + Demonstrate Option<Hash> in assurances/orderedaccumulation data #74 as reported by @davxy - there was an extra 0x20 which should not have been there since 32-byte hashes are special, see #74 for derivation, confirmed by @jaymansfield + @davxy - added SetServiceStorage lines in { assurances.txt, orderedaccumulation.txt } * fix of Incorrect items/bytes for service #82 as reported by @jaymansfield - added a_i + a_o lines in { assurances.txt, orderedaccumulation.txt } for derivation * fix of state merklization for storage as reported by @dakk - previous was 0.4.x based (TODO: check tool https://github.com/dakk/jambiato0
Alright, to increase the speed of our reasoning on what we think is the correct service storage keys + corresponding accounting, we dumped some basic logs like this:
|
In assurances/1_004 the number of items/bytes in both pre-state and post-state services are incorrect.
They should reflect the storage used by the service as mentioned in GP section 9.3 and need to be updated each time storage is changed.
The text was updated successfully, but these errors were encountered: