Chill Validation
Chill Validation
The below documents the reason help is necessary to properly regenerate the snapshot protobuf pb.go files. This error is current as of cosmos/cosmos-sdk main branch at commit 25449b5812c42e8528daee054120f2deddb3e717 ``` %...
Thank you for the review @alexanderbez Noted above is that protobuf to go code generation appears broken. Is the code generation working for your team? Would be best to have...
Works in github codespaces but not Ubuntu 20.04 x64 This is the OS I'm using for the x64 dev environment ``` NAME="Ubuntu" VERSION="20.04.4 LTS (Focal Fossa)" REPOSITORY TAG IMAGE ID...
@tac0turtle - Passing logger - good Regarding the different height, the chains in the wild have different heights because new stores are not properly added using migrations. New stores start...
Will check to see if there's a "reasonable" way to add in a check. Last check, I didn't see an obvious place to do it beautifully. It's 2023 now, so...
@tac0turtle Let's come to an agreement on what will be accepted as the goal of this PR. 1) Logging - It seems this will be allowed to merge? 2) Snapshot...
> how does the height help at the item level when it exists at the snapshot level? IAVL merkle tree roots chnage when store heights do not match. When the...
> different store versions aren't supported with in the store package today. Yes different store heights are not supported. The trouble most chains faced was due to the ability to...
> oh sorry, I confused height with version. I think it makes more sense to call height version in the snapshot. It's the nomenclature in IAVL. sorry for the confusion...
Will try to find some time later this week to finish up. @alexanderbez - Your comments overall are good (even though I have not individually responded to each of them)....