search

GEMScienceTools / gmpe-smtk

Python and OpenQuake-based Toolkit for Analysis of Strong Motions and Interpretation of GMPEs
GNU Affero General Public License v3.0
43 stars 33 forks source link

Fixing trellis plots #117

Closed rizac closed 3 years ago

rizac commented 3 years ago

This PR fixes two minor issues in trellis_plots.py:

  1. Adds to any generated dict the 'imt' key (required by eGSIM)
  2. Fixes a minor bug whereby the ylabel key of MagnitudeDistanceSpectraTrellisSigma was mapped to the y label of the superclass MagnitudeDistanceSpectraTrellis (the bug was fixed in the latter by simply calling the class _get_ylabel method, as it should be)
  3. Cleans up a lot of the module commented line, unused spaces and bad indentations which made the code less readable (unfortunately, these minor fixes make up most of the changes and hide the more important in the comparison)