debounce swallows initial value?

Question

pvakevai opened this issue 5 years ago · comments

With bacon v2.0.11

Bacon.mergeAll(Bacon.Bus().toProperty('')).toProperty().doLog('1').debounce(500).log('2')

Does not produce output that I would expect. (both 1 and 2)

Juha Paananen · Answer 1 · Tue Apr 02 2019 20:36:47 GMT+0800 (China Standard Time)

Thanks! I'll investigate this. Looks like a bug related to the "all eventstreams are now asynchronous" change in 2.0.

Matthias Seemann · Answer 2 · Tue Apr 02 2019 20:54:23 GMT+0800 (China Standard Time)

Looking at the marble diagram of debounce it is ok for it to swallow a lot of events.

source:             asdf----asdf----
source.debounce(2): -----f-------f--

Juha Paananen · Answer 3 · Tue Apr 02 2019 20:56:03 GMT+0800 (China Standard Time)

I found the bug. It's simple. Just a sec...

Juha Paananen · Answer 4 · Tue Apr 02 2019 21:10:47 GMT+0800 (China Standard Time)

The bug is not related to debounce and friends. It's mergeAll instead, passing Initial events into an EventStream, that should never happen.

Fixed in 3.0.1 and 2.0.12.

Juha Paananen · Answer 5 · Tue Apr 02 2019 21:11:04 GMT+0800 (China Standard Time)

Closing now. Please re-open if the fix doesn't help.