Skip to content

Adding a paragraph about LavaBeats in the README file. - #430

Open
JuanSMartinez wants to merge 1 commit into
android:mainfrom
JuanSMartinez:lavabeats-readme
Open

Adding a paragraph about LavaBeats in the README file.#430
JuanSMartinez wants to merge 1 commit into
android:mainfrom
JuanSMartinez:lavabeats-readme

Conversation

@JuanSMartinez

Copy link
Copy Markdown

This describes what the sample is and how it works, giving the user more context if they haven't seen the public developer documentation webpage

@alabiaga alabiaga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm with comments

The `WaveformEnvelopeBuilder` is used to construct various repetitions of these
two pulses separated by a fixed first-to-second pulse delay. The first pulse is
a chirp signal that starts at a low frequency and ends at a higher frequency
over a short duration. The second pulse is as a single period of a low frequency

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be "The second pulse is as a .."

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for catching this. Fixed

with delay in-between following a typical beats-per-minute (bpm) rate. The
result is a haptic effect that resembles a beating heart.

LavaBeats also allows to configure all the parameters of the effect to emulate

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"LavaBeats also allows users to" ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. This reads better

represented with vibration pulses of varying amplitudes and frequencies. The
first pulse represents the QRS complex of the ECG, which shows as a sharp peak
of high amplitude and short duration. The second pulse is the T wave, which has
lower amplitude, longer duration and a smoother shape.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"lower amplitude, longer duration, and a smoother shape"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added the third comma now. Thank you

This describes the sample to the users
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants