Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Option to disable rumble connection notification #149

Closed
zduny opened this issue Dec 13, 2019 · 2 comments
Closed

Option to disable rumble connection notification #149

zduny opened this issue Dec 13, 2019 · 2 comments
Labels
0 | type: enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@zduny
Copy link

zduny commented Dec 13, 2019

Is your feature request related to a problem? Please describe.
Please provide option do disable rumble effect that notifies you about successful connection to the device.

Describe the solution you'd like
An additional option in configuration menu to disable rumble connection notification (as opposed to disabling rumble altogether)

Describe alternatives you've considered
NA

Additional context
NA

@atar-axis atar-axis added the 0 | type: enhancement New feature or request label Jan 15, 2020
@atar-axis
Copy link
Owner

Good idea and should be super easy to implement. I am open to PR's :) I am too busy a.t.m

@atar-axis atar-axis added the good first issue Good for newcomers label Jan 15, 2020
kakra added a commit to kakra/xpadneo that referenced this issue May 3, 2020
According to hid-microsoft.c, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
kakra added a commit to kakra/xpadneo that referenced this issue May 3, 2020
According to hid-microsoft.c, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
kakra added a commit to kakra/xpadneo that referenced this issue May 4, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 4, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 5, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 5, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 5, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 5, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 6, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 6, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 6, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 6, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 7, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 7, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 7, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit to kakra/xpadneo that referenced this issue May 7, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: atar-axis#154
Affects: atar-axis#149
Maybe-related: atar-axis#171
Maybe-related: atar-axis#122
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit that referenced this issue May 8, 2020
According to `hid-microsoft.c`, the device supports magnitudes in the
range of 0..100.

This commit scales the magnitudes correctly from 0..65535 to 0..100 and
also uses a finer scale for the pre-calculated cosine table to have
more than 4 steps on each trigger.

Additionally, it considerably reduces the strength of the connection
notification rumble. The controller no longer tends to hop down the
table.

I don't have any specs of this but sending too high values may affect
the stability of the firmware thus this may reduce connection losses
observed in some issue reports.

This also removes the over-use of blank lines: It confuses git during
rebase as it allows splitting logical self-contained hunks into
seemingly unrelated blocks on too many opportunities.

Closes: #154
Affects: #149
Maybe-related: #171
Maybe-related: #122
Signed-off-by: Kai Krakow <[email protected]>
@kakra kakra added this to the v0.8 milestone May 29, 2020
@kakra
Copy link
Collaborator

kakra commented May 31, 2020

This has been implemented.

@kakra kakra closed this as completed May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 | type: enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants