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

forcePerTimeMeasure - Wrong #39

Open
sdeaton75 opened this issue Nov 23, 2022 · 2 comments
Open

forcePerTimeMeasure - Wrong #39

sdeaton75 opened this issue Nov 23, 2022 · 2 comments
Assignees

Comments

@sdeaton75
Copy link
Collaborator

I believe the forcePerTimeMeasure enumerations are incorrect. These appear to mostly be PressurePerTime.
forcePerTime should be something like a N/s or lbf/sec.

We need to correct this to have both UOM enumerations for PressurePerTime and forcePerTime.

All enumerations for the current forcePerTime can be moved to PressurePerTime EXCEPT GPa

@dponti
Copy link
Collaborator

dponti commented Jan 9, 2023

You're correct. Energistics 2.0 added some pressure/time units so need to harmonize the DIGGS namespace list with the new Energistics list and rename to PressurePerTimeMeasure. There is no current hardcoded property that uses the current ForcePerTimeMeasure (should be PressurePerTimeMeasure) in the current schema, so this is used for test properties at the present time.

Energistics does not list a Force per Time quantity class (eg. lbf/sec) so if we need to consider force per time. Do we have any properties that are of this quantity class? If so, we will need to create them - please provide unit definitions for what might be required (if any are required).

@sdeaton75
Copy link
Collaborator Author

We need lbf/sec and N/sec, kN/sec.

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