-
Notifications
You must be signed in to change notification settings - Fork 4
added schema for material fingerprint primary descriptors #44
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
Conversation
example/job/_project.json
Outdated
"access": { | ||
"...": "include(../security/access_legacy.json)" | ||
"type": 10, | ||
"level": 10 |
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.
We shouldnt change this
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.
Don't know the reason why this include is not working.
8.15 | ||
] | ||
], | ||
"coordNumber":[ |
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.
coordinationNumber
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.
let's use full names below too
"equilibrium": "0", | ||
"matType": "0", | ||
"vatom": 0.15 | ||
} |
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.
The above keys are not clear from first sight
"type": "array", | ||
"items": { | ||
"description": "array of s,p,d,f contributions for individual element ", | ||
"ref": "file:../../../primitive/1d_data_series.json" |
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.
Let's make this into an object?
{
's': 0.5,
'f': 0.5
}
@@ -0,0 +1,3 @@ | |||
{ |
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.
Let's try to refer to subfield of lattice_bravais?
"$ref": "file:../../../primitive/1d_data_series.json" | ||
}, | ||
"localEnvironment": { | ||
"description": "Represents the atom type each element is bonded to in the list [H, O, S, F, Cl, X]. X means an element is either non-bonded or bonded to any atom other than the atoms in the list", |
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.
Let's use enum to set the list items
@@ -0,0 +1,2600 @@ | |||
{ | |||
"elements": [ | |||
{ |
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.
Let's leave one example only
@@ -0,0 +1,15 @@ | |||
{ | |||
"type": "compound-specific", |
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.
Element-specific
"description": "structure representation.", | ||
"allOf": [ | ||
{ | ||
"$ref": "file:primary/_primary.json" |
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.
Let's include this file only once at the top?
"type": "compound-specific", | ||
"subtype": "stoichiometric", | ||
"variableType": "numeric", | ||
"dimensions": [ |
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.
values
"variableType": "numeric", | ||
"dimensions": [ | ||
{ | ||
"name": "p3", |
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.
dimensionality
"valenceOrbitalOccupancy": { | ||
"...": "include(primary/valence_orbital_occupancy.json)" | ||
}, | ||
"pNorm": { |
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.
pNorms
Let's adjust with the following in mind: type: element-specific / compound-specific OR E/C 1 E NO => schema1 {
"elements": [
{
"name": "Si",
"value": "O"
},
{
"name": "O",
"value": "X"
}
]
}
2 E OR => schema1 {
"elements": [
{
"name": "Si",
"value": "low"
},
{
"name": "O",
"value": "high"
}
]
} 3 E NU => schema1 {
"elements": [
{
"name": "Si",
"value": 1
},
{
"name": "O",
"value": 2
}
]
} 4 C NO => schema2 {
"value": "crystal"
} 5 C OR => schema2 {
"value": "low"
}
6 C NU => schema2 {
"value": 1
} 7 C NU ARRAY => schema3 {
"data": [
{
"name": "p3",
"value": 1
},
{
"name": "p2",
"value": 2
}
]
} |
"element": "Si", | ||
"valenceElectrons": { | ||
"value": 6, | ||
"name": "valence_electrons" |
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.
Remove the names
{ | ||
"id": 1, | ||
"value": "Si" | ||
"value": "Si", |
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.
We should not add into elements, into the data instead
@@ -0,0 +1,3 @@ | |||
{ |
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.
We need to discuss regarding elements schema.
Elements schema is to store elemental properties for a specific element. If I include atomic_data in elements json, I unnecessarily store the ids of the elements.
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.
OK
No description provided.