Results 95 comments of Carlos Rodriguez
trafficstars

@george-aj We have a [PR](https://github.com/cosmos/ibc-go/pull/6268) open for this issue, in case you want to take a look. I will probably merge today, but if you have any feedback, please comment...

@expertdicer Sorry for the late reply; I had missed this comment till now... Actually what needs to be done is to just remove [this code](https://github.com/cosmos/ibc-go/blob/c849a78e0df9b8cd4a658f60d5bf171161bc28fd/modules/light-clients/06-solomachine/misbehaviour_handle.go#L44-L46). The `path` should just be...

This sounds like a good idea. Thank you @AdityaSripal. > In the tendermint client, we copy over all the consensus states and update the client state according to the substitute...

I did a quick check on the number of consensus states stored for a couple of highly active channels: - [For the light client on Osmosis used in the channel...

Pasting here some feedback from @tac0turtle on Slack: "In feegrant we did two things. Iterate 200 grants for pruning. If there are more users can submit a tx to prune...

The changes would basically be: ```diff diff --git a/modules/apps/29-fee/ibc_middleware.go b/modules/apps/29-fee/ibc_middleware.go index 4ac1e7dd9..13f92c644 100644 --- a/modules/apps/29-fee/ibc_middleware.go +++ b/modules/apps/29-fee/ibc_middleware.go @@ -429,6 +429,9 @@ func (im IBCMiddleware) OnChanUpgradeOpen(ctx sdk.Context, portID, channelID str versionMetadata,...

Wondering if we need to add this to spec... Currently the spec doesn't really specify any possible encoding formats and seems to leave this to the implementations... what do you...

@dreamer-zq do you have answer to this question?

Thank you for submitting this new spec, @liangping. I will try to give it a first review this quarter, but realistically you should expect that it gets merged somewhere in...

Hi @michwqy. Thanks for bringing up these questions! **Question 1** > note: the connection may have been closed I think the comment is not correct and can be removed, [`CLOSED`...