Document noise reduction module

[duyquangnguyenhac]

Fixes #1170 (<=== Replace 0000 with the Issue Number)

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with npm test
• code is in uniquely-named feature branch and has no merge conflicts
• PR is descriptively titled
• ask @publiclab/is-reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

[welcome]

Thanks for opening this pull request!
There may be some errors, but don't worry! We're here to help! 👍🎉😄

[codecov]

Codecov Report

Merging #1189 into main will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1189   +/-   ##
=======================================
  Coverage   32.07%   32.07%           
=======================================
  Files         107      107           
  Lines        1986     1986           
  Branches      297      297           
=======================================
  Hits          637      637           
  Misses       1349     1349

[aashna27]

Great work

[Divy123]

Looks great to me. Just a few comments and this is done.

[duyquangnguyenhac]

Hi, I've fixed the documentation as suggested, please review it, thank you!

[aashna27]

The conflict needs to be resolved and then this can be merged.

[duyquangnguyenhac]

Hi, the conflict is resolved. Please check it out!, Thank you.

[aashna27]

Hi, the conflict is resolved. Please check it out!, Thank you.

Sorry to point out again but the conflicts continue to exist.

[duyquangnguyenhac]

Hi, the conflict is resolved. Please check it out!, Thank you.

Sorry to point out again but the conflicts continue to exist.

I have merged main and resolved the conflicts.

[duyquangnguyenhac]

Hi, please approve this issue.
Thank you, James.

[duyquangnguyenhac]

Hi @aashna27, I've resolved all comments and I believe my PR is ready to merge. Can you help me getting this merged? I don't want to leave this PR open for too long.

[aashna27]

Yea sure i ll help you

…

On Thu, Sep 12, 2019, 4:16 AM duyquangnguyenhac ***@***.***> wrote: Hi @aashna27 <https://github.com/aashna27>, I've resolved all comments and I believe my PR is ready to merge. Can you help me getting this merged? I don't want to leave this PR open for too long. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1189?email_source=notifications&email_token=AGKQ7SGPLE42O22ZUGMWBH3QJFYNPA5CNFSM4IHOP7IKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6QD22Q#issuecomment-530595178>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGKQ7SAMUCEXN7J4JFMYUEDQJFYNPANCNFSM4IHOP7IA> .

[aashna27]

Please add it alphabetically and then we can merge this

[aashna27]

Thanks for your help

[duyquangnguyenhac]

Module is now alphabetically sorted!

[duyquangnguyenhac]

Hi,
I think it is okay to merge now?

• Thank you so much for supports!
  James

[jywarren]

Looks awesome. I've resolved conflicts one more time, then we will merge! Thanks for your patience!

[harshkhandeparkar]

Idk if I have left an official review here, but I have gone through it. So consider it as a 👍 from me. Thanks!

…

On Mon, 23 Sep, 2019, 8:04 PM Jeffrey Warren, ***@***.***> wrote: Looks awesome. I've resolved conflicts one more time, then we will merge! Thanks for your patience! — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1189?email_source=notifications&email_token=AIJI5H72J5KM4VIQRDA4K2DQLDHWNA5CNFSM4IHOP7IKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7LCBDQ#issuecomment-534126734>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AIJI5HZH4IJXC6I572R4D2DQLDHWNANCNFSM4IHOP7IA> .

[welcome]

Congrats on merging your first pull request! 🙌🎉⚡️
Your code will be published to https://beta.sequencer.publiclab.org in a day or two.
In the meantime, can you tell us your Twitter handle so we can thank you properly?
Now that you've completed this, you can help someone else take their first step!
See: Public Lab's coding community!

[jywarren]

Awesome! Thanks a ton!