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

Leakage test #75

Closed
wants to merge 41 commits into from
Closed

Leakage test #75

wants to merge 41 commits into from

Conversation

selcan-k
Copy link
Contributor

@selcan-k selcan-k commented Jul 4, 2024

Description

An Ontology for a leakage test.

Type of change

Please delete options that are not relevant.

  • New type (non-breaking change which adds a type)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any
relevant details for your test configuration

  • Test A
  • Test B

Checklist

  • My code follows the modelling guidelines of this project
  • I have performed a self-review of my own changes

Copy link
Contributor

@lenasauermann lenasauermann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All variables names for classes and varables must be in english.
The name of the classes must be in PascalCase. Example: Sensor
The attributes of the classes must be in camelCase. Example: sensorType

MaschinenparameterShape and ProzessparameterShape are missing.

#57 Please add Ontology-Class for every Shape as described in this issue.

robertschubert and others added 23 commits July 9, 2024 09:36
Signed-off-by: GitHub Action <[email protected]>
Signed-off-by: Sebastian_3DMS <[email protected]>
Signed-off-by: selcan-k <[email protected]>
* added new correction from sh:class to sh:node (reason see README.md)

* Add VARIABLES.md files

Signed-off-by: GitHub Action <[email protected]>

---------

Signed-off-by: GitHub Action <[email protected]>
Co-authored-by: schuberr <[email protected]>
Co-authored-by: GitHub Action <[email protected]>
Files for Leakage Test

Signed-off-by: selcan-k <[email protected]>
Signed-off-by: GitHub Action <[email protected]>
Signed-off-by: selcan-k <[email protected]>
Signed-off-by: GitHub Action <[email protected]>
Signed-off-by: selcan-k <[email protected]>
Signed-off-by: GitHub Action <[email protected]>
Signed-off-by: selcan-k <[email protected]>
Signed-off-by: GitHub Action <[email protected]>
Signed-off-by: selcan-k <[email protected]>
Signed-off-by: GitHub Action <[email protected]>
Signed-off-by: selcan-k <[email protected]>
… and stored in the GITHUB-ENV. (#74)

Also Bugfix when having several instance files to be checked: Add them together into the data graph since they might reference each other.

Co-authored-by: schuberr <[email protected]>
selcan-k and others added 17 commits July 9, 2024 09:39
* added details to modeling guidelines and known issues

Signed-off-by: Lena Sauermann <[email protected]>

* fix review comments

Signed-off by: Lena Sauermann <[email protected]>

* fixes based on review comments

Signed-off-by: Lena Sauermann <[email protected]>

* remove comment about instance files

Signed-off-by: Lena Sauermann <[email protected]>

---------

Signed-off-by: Lena Sauermann <[email protected]>
Co-authored-by: schuberr <[email protected]>
Signed-off-by: GitHub Action <[email protected]>
Signed-off-by: GitHub Action <[email protected]>
@selcan-k selcan-k closed this Jul 9, 2024
@selcan-k selcan-k deleted the leakage-test branch July 9, 2024 08:17
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

Successfully merging this pull request may close these issues.

6 participants