lsst / rubin_sim

Scheduler, survey strategy analysis, and other simulation tools for Rubin Observatory.
https://rubin-sim.lsst.io
GNU General Public License v3.0
41 stars 37 forks source link

OPSIM:1050 Update m52snr and add y-band parallax metric #360

Closed rhiannonlynne closed 1 year ago

rhiannonlynne commented 1 year ago

Many of the commits in this PR are updating line length in doc strings, moving a bit more code to snake_case from camelCase, removing unnecessary or unused code (imports or values that were not used), updating super() calls to remove the / self arguments, or standardizing metric API ("Metric" at the end of metric classnames, metric_name doesn't need to be a kwarg if the default is the same as the class name).

The meaningful changes though are:

and the bigger changes: