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

Hz vs rad/sec #208

Closed
Bob-May opened this issue Aug 30, 2023 · 3 comments
Closed

Hz vs rad/sec #208

Bob-May opened this issue Aug 30, 2023 · 3 comments

Comments

@Bob-May
Copy link

Bob-May commented Aug 30, 2023

EngineeringPaper.xyz treats any result with a dimension of 1/Time as Hz.

However rad/sec also has the dimension of 1/Time (specifically, 1/Sec)

The conversion between Hz and rad/sec is 2pi rad/cycle.

I cannot find any way to properly introduce this conversion.

See: https://engineeringpaper.xyz/iM5tvNfmbVbRgB2Y4uynGb

@Bob-May
Copy link
Author

Bob-May commented Aug 30, 2023

I see that you already know about this flaw. https://blog.engineeringpaper.xyz/calculating-the-natural-frequencies-of-a-cantilever-beam

Since EngineeringPaper.xyz will always interpret [1/sec] as being in Hertz, special care always needs to be taken with frequency calculations to explicitly force them to be in Hertz or to be in radians per second.

The point of using "units aware" calculation tools is to avoid the need to take "special care" with units - the tool is supposed to take care of this for you.

Sort of needs to be fixed, don't you think?

@mgreminger
Copy link
Owner

mgreminger commented Aug 31, 2023

Thanks for the feedback. There's no way to automatically convert between radians/sec and Hz since radians would need to be interpreted as a unitless $2\pi$ rather than it's normal interpretation as an angular measurement. MathCad handles this situation in the same way. In addition to the approach you identified of dividing by $2\pi$, there are several options for making the conversion:
Screenshot from 2023-08-31 11-25-21

mgreminger added a commit that referenced this issue Aug 31, 2023
Prevent regressions in angular frequency conversions as discussed in #208
mgreminger added a commit that referenced this issue Nov 17, 2023
Partially addresses the confusion raised in #208
@mgreminger
Copy link
Owner

Closing since #217 prevents [1/s] from being automatically converted to [Hz], which addresses some of the issue raised here. In general, the conversion between [rad/sec] and [Hz] needs to be handled using one of the approaches outlined above.

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