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

docs: Added link to clean air calculation pdf #68

Merged
merged 4 commits into from
Dec 21, 2020
Merged

Conversation

angrox
Copy link
Collaborator

@angrox angrox commented Nov 24, 2020

Added link to the theory pdf.

@angrox angrox marked this pull request as ready for review November 24, 2020 18:42
@angrox angrox requested review from mlkiefer and thorsten November 24, 2020 18:42
documentation/README.md Outdated Show resolved Hide resolved
README.md Outdated
@@ -30,6 +30,8 @@ Here are some reading information sources, if you want to read up on various opi
- https://www.cuimc.columbia.edu/news/far-uvc-light-safely-kills-airborne-coronaviruses
- https://pubmed.ncbi.nlm.nih.gov/32547908/

We also have a [calculation (PDF)](documentation/theory.pdf) of the theoretical effiency of the clean air device.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
We also have a [calculation (PDF)](documentation/theory.pdf) of the theoretical effiency of the clean air device.
We estimated the irradiance inside the device as described in the [calculation (PDF)](documentation/theory.pdf).

@@ -4,6 +4,10 @@ The documentation covers all information around the UV-C clean air units.

## DIY Hardware

### Calculations

The effiency calculation can be found [here](theory.pdf).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The effiency calculation can be found [here](theory.pdf).
Estimations about the irradiance and considerations about the room size for which such a device could be feasible are located [here](theory.pdf).

Copy link
Contributor

@mlkiefer mlkiefer left a comment

Choose a reason for hiding this comment

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

I've added some suggestions

@angrox angrox requested a review from thegreatdane6 December 17, 2020 07:22
Co-authored-by: mlkiefer <[email protected]>
@angrox angrox dismissed stale reviews from mlkiefer and thorsten December 21, 2020 12:11

no.

@angrox angrox merged commit 3bc703b into main Dec 21, 2020
@thegreatdane6 thegreatdane6 deleted the add_theory_pdf_link branch December 22, 2020 09:41
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.

4 participants