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

Wrong conversion between rpm and rad/s #373

Closed
elaye opened this issue Feb 22, 2024 · 3 comments
Closed

Wrong conversion between rpm and rad/s #373

elaye opened this issue Feb 22, 2024 · 3 comments

Comments

@elaye
Copy link

elaye commented Feb 22, 2024

Hello,

First of all thank you for this project, it's really useful and very well made!

I have an issue with the conversion between rpm and rad/s. When I try the following conversion 1 rad/s -> rpm numbat gives me 60 RPM even though it should output 9.5493. It seems there is a factor missing according to the wikipedia article.

@sharkdp
Copy link
Owner

sharkdp commented Feb 22, 2024

First of all thank you for this project, it's really useful and very well made!

Thank you for the feedback!

I have an issue with the conversion between rpm and rad/s

This is a common question that already came up a few times in the predecessor project. You might be interested in reading through those discussions (sharkdp/insect#187 or sharkdp/insect#50). The fundamental problem lies with the International system of units (SI), not with how it's defined in Numbat. The definition of rad and Hz in the SI lead to inconsistencies.

There is a way to resolve those inconsistencies, but it would require a change to the SI. Angles would need to have their own physical dimension. We actually do have support for this in Numbat, but it's currently only active on a feature branch. You can read (much) more about this here: #167 (see also the RPM example in the description). And you can try a (slightly outdated) version of Numbat with dimensionful angles here: https://numbat.dev/angles/. It will give you the correct answer that you expected.

@sharkdp
Copy link
Owner

sharkdp commented Mar 9, 2024

@elaye Any feedback on this?

@elaye
Copy link
Author

elaye commented Mar 10, 2024

Thank you for your answer, that makes a lot of sense even though it's counter-intuitive!

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

No branches or pull requests

2 participants