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

remove TestResult from (Resource)PropertyType #44

Closed
racheliurui opened this issue Sep 17, 2019 · 3 comments
Closed

remove TestResult from (Resource)PropertyType #44

racheliurui opened this issue Sep 17, 2019 · 3 comments

Comments

@racheliurui
Copy link
Contributor

Background

ISA-950002 changed Test Results to a 1st order object as part of the Operations Test Model. Test Result is a 1st order object is able to be exchanged/queried and has a relationship to resource actual which is represented as a root attribute

But current B2MML 7.0 still has the TestResult type as an attribute in each instance resource property. The old resource model had a composite relationship for resource properties which did not allow Test Result attribute to be exchanged and queried as a 1st class object.

Supporting Document

ISA-950002

Section 5.9.4
Table 104 Test specification attributes
Table 45 – Equipment property relationships
Table 46 – Equipment property attributes
Table 70 – Material lot property relationship roles
Table 71 – Material lot property attributes
Table 55 – Physical asset property relationship roles
Table 56 – Physical asset property attributes
Table 36 – Person property relationship roles
Table 37 – Person property attributes

Impacted Types

B2MML - Equipment.xsd

  • EquipmentPropertyType
    B2MML - Material.xsd
  • MaterialLotPropertyType
    B2MML - PhysicalAsset.xsd
  • "PhysicalAssetPropertyType"
    B2MML - Personnel.xsd
  • "PersonPropertyType",
@Dennis-Brandl
Copy link
Contributor

I have no idea what is being proposed here.

@gerardmoloneyetpartners

Hi Dennis, We will update the issue description to make it clear.

@Dennis-Brandl
Copy link
Contributor

To be included in the soon to be released version. Issue closed.

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

3 participants