tendermint

Summary

We should charge extra gas when doing serialization.

Epic: #8917

Problem Definition

Today we only charge gas for doing storage operations and loops. For simple operations we only charge gas for a known expensive operations, like signature validation. Serialization is also known to be expensive (especially the "text mode" like JSON). And if there is lot of back and forth t

It wouldn't start if genesisPath and GCI is specified.

Node v11.8.0

Lotion 0.4.8

`
lotion({
initialState: {
blockNo: 0,
txCount: 0
},
ports: { abci: 58354, p2p: 58355, rpc: 58356 },
GCI:
'08a5972afaf42e5751cd7f1af542c4e5e259e07dd0d45d31ffba3eaced632549',
genesisPath:
'/Users/admin/.lotion/networks/806f53597c1bfb8fdd9d128aca662a19/config/genesis.json'
})

e.g. https://github.com/crypto-org-chain/chain-main/blob/0f99b253887b4bcce0dac2fd65635fb07c00920b/x/nft/keeper/keeper_test.go#L52

it may be worth switching to https://github.com/onsi/ginkgo

Since raw queries only return encrypted keys, it's pointless and will break implementations coming from other networks without letting anyone know.

We should remove it from the next version

Problem Definition

We hardcode date formatting in all places. We should have a single function to make sure we format date consistently.
Ref: regen-network/regen-ledger#647 (comment)

Proposal

func FmtDate(t time.time) string {
    return t.Format("2006-01-02")
}

For Admin Use

• Not duplicate issue
• Appropriate

This is self-explanatory: celestiaorg/celestia-core#401 (review)

MsgDeleteCoordinator is used to explicitly declare the coordinator as no longer active. However, we should not remove coordinator attached information for documentation purposes (the information can be retrieved through chain history anyway)