Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • S shakyground2
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 11
    • Issues 11
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Shakemap
  • shakyground2
  • Merge requests
  • !28

Small edit to adapt to OQ changes, fixes broken quakeml test

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Graeme Weatherill requested to merge gweather/shakyground2:bug/oq_update_fix into master Oct 04, 2021
  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 2

In the last few months OQ has made some API changes to its ground motion model library, as explained here: #25. This has caused some breakages in this library (and other libraries that use OQ's GMMs for this purpose). At present, shakyground2's shakemap calculator is using the openquake.hazardlib.contexts.ContextMaker class for preparing the inputs to the GMM, but calling the GMMs themselves according to the old style API. After some user feedback, OpenQuake's developers have ensured that OpenQuake's old style GMM remains backward compatible for the time being, so only one small change was needed to keep shakyground2 working with the 3.12.0 release of OpenQuake. This is implemented here along with some minor cleanup of formatting to address a line too long formatting error from Flake8.

Also, a small change in the precision of the magnitude and location of one of the GEOFON test events seemed to have occurred, so the test have been edited to fix this too.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: bug/oq_update_fix