@TxCorpi0x reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @masihyeganeh and @ysv)

x/pse/keeper/final_score_map.go line 42 at r1 (raw file):

	key, err := m.addressCodec.BytesToString(addr)
	if err != nil {
		return

Are we should that BytesToString is always no-error in our case? if yes, i think it would be beneficial to have a wrapper without returned type for address conversion. like MustAccAddressFromBech32 in the prior design.

@ysv reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @masihyeganeh and @TxCorpi0x)

x/pse/keeper/final_score_map.go line 42 at r1 (raw file):

yes
I tend to agree that it doesn't look correct to just return if err is present here

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @masihyeganeh and @TxCorpi0x)

x/pse/keeper/final_score_map.go line 42 at r1 (raw file):

blocking

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @masihyeganeh, @TxCorpi0x, and @ysv)

x/pse/keeper/final_score_map.go line 42 at r1 (raw file):

so we should panic instead of returning an error ?

@TxCorpi0x reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @masihyeganeh and @ysv)

@ysv reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @masihyeganeh and @TxCorpi0x)