Canterbury and Wellington are always Type 4 and 3 by default#10
Canterbury and Wellington are always Type 4 and 3 by default#10sungeunbae wants to merge 1 commit intomainfrom
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request standardizes the Highlights
Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates the nzcvm_registry.yaml file to add a type attribute to several historical Canterbury and Wellington basin models. The changes set type: 4 for Canterbury_v18p1, Canterbury_v18p2, and Canterbury_v18p3, and type: 3 for Wellington_v19p1 and Wellington_v19p6. These changes align with the types of more recent versions of these basins and are consistent with the PR's goal. The changes are valid according to the project's schema.
There was a problem hiding this comment.
Pull request overview
This PR updates the NZCVM dataset registry to add missing basin type metadata for several existing basin versions, improving consistency with the registry schema and downstream tooling that surfaces basin type information (e.g., wiki generation).
Changes:
- Added
type: 4to Canterbury basin versionsCanterbury_v18p1,Canterbury_v18p2, andCanterbury_v18p3. - Added
type: 3to Wellington basin versionsWellington_v19p1andWellington_v19p6.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Canterbury - Type 4 (since the start) |
|
This is the exact comment: "Canterbury has always been Type 4 as long as we've run production quality simulations back in 2016/2017 with only minor changes within those 2 years. Wellington has changed from one model to another. The suffix indicates the year and month. e.g. 19p1 is January of 2019. v19p1 and/or v19p6 is Semmens (2010) or something similar. I suggest that the remainder of the basins, which do not have a type specified, be corrected. I believe this really only happens for Wellington and Canterbury where some of the past basins do not have a type specified. This came about as I was preparing the basin identifications for NZCVM v2.06 which was used in CyberShakeNZ 200m and 100m. Whichever Canterbury and Wellington basin versions were used by this NZCVM version had nans. |
Aaron consulted Robin Lee and confirmed all historical Canterbury and Wellington basins are of type 4 and 3 respectively.