Roman

Results 85 comments of Roman

This is definitely a weird case that was spotted and thanks for bringing it up! I agree with the conclusion From my understanding, the intent was to make sure that...

> I have a weird case that goes like this: if I set a gauge with start time = 5, and the current block time = 4, then `GetModuleToDistributeCoins` will...

> u mean like [this](https://github.com/osmosis-labs/osmosis/commit/293198d5401b7d8c5db18f3a6128df6e9dbdb03c) I think that's what Matt meant. Have you gotten a chance to try it? Also, @catShaark could you please give me the branch where you...

@catShaark Thank you for your patience and for giving me the branch. So I saw the following log when I ran your test: ``` gauge start time: 123456799 block 1...

That being said, I don't understand who are the consumers of this query and what value exactly it provides. Do we even want to have it?

> > That being said, I don't understand who are the consumers of this query and what value exactly it provides. Do we even want to have it? > >...

That would be great! Please do when you get a chance. That way, we can also get feedback from more people on this as well!

Yes, I will work on this. Will address: https://github.com/osmosis-labs/osmosis/issues/2367 first to easily update both `main` and `v11.x`

> The proto docker image should be updated later today. > > I noticed we're using super-linter-- fyi we dropped that from pylons; it kept giving us pretty poor results....

hey @hieuvubk could you please resolve the conflicts? We can merge this right after