-
Notifications
You must be signed in to change notification settings - Fork 54
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
INCODE Application Areas data models #144
Conversation
|
if you need support please let me know or book a support session here |
examples lower case - deleted png files - added some missing examples
I have made some changes in the latest commits to the data models of the different Application Areas (AA1 - AA2 - AA3 - AA4)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check the same errors you have in the first two models across the rest of them. And eventually try to validate first with the testing service before contributing
https://smartdatamodels.org/index.php/data-models-contribution-api/
thanks
"engineStatus": 1, | ||
"phi": "37.8745952", | ||
"lamda": "23.921456", | ||
"z": "87", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
@@ -0,0 +1,96 @@ | |||
{ | |||
"$schema": "http://json-schema.org/schema#", | |||
"$schemaVersion": "0.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
0.0.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am approving the PR because I need to for fixing a problem. but the changes needs to be addressed anyway
Thank you Alberto, I will check your comments and make the necessary changes that I can do. (I will still do it on this branch, so i will not delete it yet) |
All the comments are resolved (in a new pull request: #146) |
I have added all the data models produced from the INCODE Application Areas (AA1, AA2, AA3, AA4)
Next steps: