-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Schema] should taxonomy be optional in vulnerability? #202
Comments
Good point - some vulnerability relationships relate to only a general occupancy type e.g. 'residential', 'commercial', 'industrial'. Either we (1) assign taxonomy type based on this - we can create one in ODS/GEM taxonomies for general residential with all other taxonomy string components as unknown, or (2) make taxonomy optional and fall back on the occupancy type only - which now I look for it, isn't within the schema or codelists (was this removed, we had it in at one point). We need some way to tie the V relationships to a type of exposure - and using |
We did remove occupancy type as this isn't necessarily always the same across every asset in a dataset.
|
Taxonomy should be optional, because exposure grouping is often custom.
That is true, yet datasets could include one or more of these occupancy types. Right now we identify only |
Not having this in the metadata reduces search capability - it is useful for users searching V functions to filter by those suitable for Residential buildings, or by a certain construction type, for example (see OpenQuake tool example), but yes by including in metadata we potentially introduce long string / array of occupancy types |
For this iteration of the standard I think we should make it optional. The next version of the standard could look to work out if occupancy is worth putting back in, and in conjunction making |
@stufraser1 are you okay with the above suggestion? |
From example being developed in #135 (comment) the dataset doesn't have a specific taxonomy scheme that it's used. Should we make this field optional instead of required?
If it does need to stay as required we should add a code to 'classification_schema.csv' to cover these scenario's for consistency and add a bit to the guidance to state what to use in this situation.
The text was updated successfully, but these errors were encountered: