khanh

> How can an "upcoming gauge" have `start_time` > > > I thought the upcoming gauge are, gauges whose `start_time` has not yet occurred Because gauges are initially set as...

> aah i see so you're talking about this case; > > ``` > Epoch time: 1, Start time: 3 > When Epoch = 4 then, Start time < Epoch...

So we gotta fix the function right, or is this intended ?

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 count...

@p0mvn, plz checkout this [test](https://github.com/osmosis-labs/osmosis/blob/bef92048011ed463af4f2d2ef9b633a8ebfdf076/x/incentives/keeper/distribute_test.go#L188) I wrote.

u mean like [this](https://github.com/osmosis-labs/osmosis/commit/293198d5401b7d8c5db18f3a6128df6e9dbdb03c)

https://github.com/osmosis-labs/osmosis/compare/weird-case-getmoduletodistributecoin, here is the branch. Also this is the test's name TestWeirdCaseGetModuleToDistributeCoins

> @catShaark Thank you for your patience and for giving me the branch. > > So I saw the following log when I ran your test: > > ``` >...

> 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? It's only used...

yess, I totally agree with you, should I create a PR for applying `k.UpcomingGaugesIterator(ctx)`