dotnet / dotnet/reactive

Debounce incorrectly named throttle

Open
#1,471 4 comments 2 reactions 0 assignees View on GitHub
[area] Rx
Dominant language
C#
Stars
7.2k
Forks
798
PR merge metrics
No merged PRs in 30d

Description

#### Bug

> Which library version?

Latest

> What are the platform(s), environment(s) and related component version(s)?

All

> What is the use case or problem?

The Debounce is still called Throttle in System.Reactive, and the Throttle is somewhat achieved by sample (reopens https://github.com/dotnet/reactive/issues/395).

> What is the expected outcome?

The names for these things should be how they are everywhere else.

> What is the actual outcome?

For some reason, Debounce was called Throttle in C#.

> Do you have a code snippet or project that reproduces the problem?

To a certain extent, elaborating on what debounce and throttle mean and do might only add confusion to this ticket as it did in the last. These snapshots from the documentation should illuminate the issue:

Capture

**From Reactivex.io**. Observe RxNET calls Throttle what is called in plurality Debounce.

Additionally, the behavior that is tested [here](https://github.com/dotnet/reactive/blob/main/Rx.NET/Source/tests/Tests.System.Reactive/Tests/Linq/Observable/ThrottleTest.cs) is a debounce. It is similar to the tests in the [Java implementation](https://github.com/ReactiveX/RxJava/blob/3.x/src/test/java/io/reactivex/rxjava3/internal/operators/observable/ObservableDebounceTest.java).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.