search

DIGGSml / schema-dev

Development repository for DIGGS Schema Files. The latest updates are found here.
Mozilla Public License 2.0
19 stars 5 forks source link

energyDensity UOM Issue #43

Open sdeaton75 opened 1 year ago

sdeaton75 commented 1 year ago

We have an issue with the energyDensity element where it's uom attribute doesn't seem to be valid. The units of measure are in the kernel for it, but I don't think it is included in the union of all of the uom values. No matter which of the uom values we input, it won't validate the file.

image

uom for energy density below. image

dponti commented 1 year ago

UnionAll is fixed in 2.6-dev. energyDensity is (I think) an inappropriate name for this property - it should be mechanicalSpecificEnergy. (?)

sdeaton75 commented 1 year ago

Can we fix this in 2.5 also, so we don't get invalid DIGGS files when including this property?

AC107 commented 1 year ago

Added to list

— Allen Cadden / Principal

From: Scott Deaton @.> Sent: Monday, January 9, 2023 6:40 AM To: DIGGSml/diggs-schema @.> Cc: Subscribed @.***> Subject: Re: [DIGGSml/diggs-schema] energyDensity UOM Issue (Issue #43)

Can we fix this in 2.5 also, so we don't get invalid DIGGS files when including this property?

— Reply to this email directly, view it on GitHubhttps://github.com/DIGGSml/diggs-schema/issues/43#issuecomment-1375499344, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGJD4COX4FMA4IABE4R3FHLWRP2IJANCNFSM6AAAAAAS5GGDEU. You are receiving this because you are subscribed to this thread.Message ID: @.**@.>>