-
Notifications
You must be signed in to change notification settings - Fork 390
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
Add mega force and torque #342
Add mega force and torque #342
Conversation
Thanks I'm glad you appreciate it and that you decided to help out improving it! |
Glad you haven't time to do it too fast. I just saw that in pounds units doesn't show up (AbbreviationsWithPrefixes are missing). I fix it and commit asap. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks! Verified the test case numbers.
"Localization": [ | ||
{ | ||
"Culture": "en-US", | ||
"Abbreviations": [ "lbf·in" ], | ||
"AbbreviationsWithPrefixes": [ "kipf·in" ] | ||
"AbbreviationsWithPrefixes": [ "kipf·in", "Mlbf·in"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to me that kipf
or kip
is a special case of widely used abbreviations although it "should" have been named klbf·in
for consistency. Unfortunately the current code generator doesn't support multiple units per prefix, so we have to choose one and I won't change it now to avoid breaking anything. I'm creating a new issue to address the need to support multiple abbreviations for prefixed units so we can add it later. See #344
You don't have to take any action in this PR, this is just an observation on my part.
Hi Andreas,
First of all, thank you very much for your wonderful work with UnitsNet. It is really helpful!
I need for one of my project bigger units for torques and forces. I have followed your instruction to add them and send you this mergerequest.
I am a beginner with github so tell me if I have done something wrong.
Best regards,
Christophe