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

feat: ppm parts per million #544

Merged
merged 1 commit into from
Jan 20, 2024
Merged

feat: ppm parts per million #544

merged 1 commit into from
Jan 20, 2024

Conversation

nebkat
Copy link
Contributor

@nebkat nebkat commented Jan 20, 2024

No description provided.

Copy link
Collaborator

@JohelEGP JohelEGP left a comment

Choose a reason for hiding this comment

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

For reference, the SI brochure mentions:

5.4.7 Stating quantity values being pure numbers
[...]
The term “ppm”, meaning 10⁻⁶ relative value, or 1 part in 10⁶, or parts per million, is also
used. This is analogous to the meaning of percent as parts per hundred. The terms “parts per
billion” and “parts per trillion” and their respective abbreviations “ppb” and “ppt”, are also
used, but their meanings are language dependent. For this reason the abbreviations ppb and
ppt should be avoided.
[In a side note:]
In English-speaking
countries, a billion is now
generally taken to be 10⁹
and a trillion
to be 10¹²; however,
a billion may still
sometimes be
interpreted as 10¹²
and a trillion as 10¹⁸.
The abbreviation ppt
is also sometimes read
as parts per thousand,
adding further confusion.

src/core/include/mp-units/unit.h Outdated Show resolved Hide resolved
test/static/unit_symbol_test.cpp Outdated Show resolved Hide resolved
Co-authored-by: Mateusz Pusz <[email protected]>
Copy link
Owner

@mpusz mpusz left a comment

Choose a reason for hiding this comment

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

Thanks!

@mpusz mpusz merged commit 6e8a21a into mpusz:master Jan 20, 2024
45 checks passed
@nebkat nebkat deleted the patch-1 branch January 20, 2024 17:32
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.

3 participants