UnitsNet icon indicating copy to clipboard operation
UnitsNet copied to clipboard

Add quantity for Impulse (integral of force over time)

Open Sigvess92 opened this issue 2 years ago • 2 comments

We are using UnitsNet to handle force plate data and derived metrics. We utilize quantities such as Force, Torque and ForceChangeRate, but have not been able to figure out what to do with impulse data (integral of force over time). Creating and maintaining our own impulse type is not optimal, especially if it could also be of use to others by adding it to the main repo.

Does a suitable quantity for impulse exist already under a different name? Is there a reason why such a quantity doesn't/shouldn't be a part of UnitsNet or can it be added? I could contribute with a PR on this request if accepted.

Sigvess92 avatar Sep 20 '22 12:09 Sigvess92

Did you check https://github.com/angularsen/UnitsNet/wiki/Adding-a-New-Unit for further information?

Brunni avatar Sep 22 '22 10:09 Brunni

Did you check https://github.com/angularsen/UnitsNet/wiki/Adding-a-New-Unit for further information?

Yes, and it seems to me that it should tick all the boxes in the "A quantity is a good fit to add" section. Did you mean to refer to a specific point in the document?

Sigvess92 avatar Sep 22 '22 11:09 Sigvess92

@Sigvess92 Sorry for the late reply, I have been missing out on email notifications for some time.

I think Impulse is a great fit, please go ahead with the PR.

What units do you propose?

I see some common ones here: https://www.translatorscafe.com/unit-converter/en-US/impulse/

angularsen avatar Oct 31 '22 22:10 angularsen

@angularsen Great! Will hopefully get it prioritized soon.

Newton seconds is by far the most common unit, but it is probably a good idea to include everything in the source you linked.

Sigvess92 avatar Dec 01 '22 14:12 Sigvess92

@angularsen Created PR: https://github.com/angularsen/UnitsNet/pull/1178

I decided to only add the units that were used in systems of quantities relevant to the field of mechanics. The site you linked also included unrelated but dimensionally equivalent units which I decided to skip.

Sigvess92 avatar Jan 04 '23 14:01 Sigvess92

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 18 '23 14:06 stale[bot]

Merged by #1178

angularsen avatar Jun 19 '23 11:06 angularsen