-
Notifications
You must be signed in to change notification settings - Fork 8
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
[Property] OED fields update for offshore assets modelling #191
Comments
Thanks @jm-moratalla Have reviewed and have the following comments
Reason for 2: you may not always know if soil is liquefiable or it may not be relevant (for non earthquake locations). If it is unknown then modeller can choose how to map the exposures to vulnerability given the uncertainty. |
Thanks Jo.
We have proposed FoundationType 13 onwards, as they relate to offshore, so have provided notes for these. But we have not proposed the onshore foundations, so it perhaps wouldn’t be appropriate for us to propose the notes. As an idea, can these be mostly taken from the GEM taxonomy (i.e. could we perhaps ask someone from GEM to have a look at the onshore ones (i.e. copy across the notes/descriptions from the GEM taxonomy for the most relevant foundation types)?
Re your suggestions on SoilLiquefiable, yes these changes are absolutely fine with us.
Jose Moratalla
Catastrophe Model Developer
E: ***@***.***
[A logo with blue and black text Description automatically generated]
From: johcarter ***@***.***>
Sent: Wednesday, June 19, 2024 12:06 PM
To: OasisLMF/ODS_OpenExposureData ***@***.***>
Cc: Jose Moratalla ***@***.***>; Mention ***@***.***>
Subject: Re: [OasisLMF/ODS_OpenExposureData] OED fields update for offshore assets modelling (Issue #191)
Thanks @jm-moratalla<https://github.com/jm-moratalla>
Have reviewed and have the following comments
* As you are changing descriptions only and adding a new field and codes, this would be a minor non-breaking change but would go into the next major OED release v4 as that is the next release.
* You have sent notes for FoundationType 13+, but can you also supply helpful notes for 1-12 which are currently N/A?
* New field SoilLiquefiable - a few suggestions
1. Make data type tinyint with default value 0
2. Make default value 0 mean unknown / or not applicable and shift no and yes to values 1 and 2
Reason for 2: you may not always know if soil is liquefiable or it may not be relevant (for non earthquake locations). If it is unknown then modeller can choose how to map the exposures to vulnerability given the uncertainty.
—
Reply to this email directly, view it on GitHub<#191 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BEL7QMZATJY2B3XAKCVRGZDZIFJYNAVCNFSM6AAAAABJRUDKOGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZYGI4DKOBTGI>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
|
Under discussion (new field) fyi @aiste-kalinauskaite |
OpenExposureData_Spec-RenewRisk.Proposal.240618_updated.xlsx Updated proposal as discussed above. Will be implemented if there are no further comments. |
Implemented version attached OpenExposureData_Spec-RenewRisk.Proposal.240618_updated2.xlsx |
All looks fine - no additional comments from me. |
Description
We are proposing updates to the OED schema, to allow for coding of offshore renewable assets.
See attached .xlsx for proposed changes.
Cells highlighted yellow, have been edited/added from the original schema.
Unedited rows/tabs have been removed for clarity.
Reasons for change
To allow for coding of offshore renewable assets.
Scope of change
Changes proposed to:
Impact of change
These changes should only impact models by Renew-Risk, as other models do not yet use the proposed codings, to our knowledge.
Data type, default values, are blanks allowed, list valid values
See attached .xlsx for proposal.
OpenExposureData_Spec-RenewRisk Proposal 240618.xlsx
The text was updated successfully, but these errors were encountered: