search

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-1033 : Add GeneralUniformityMetric #327

Closed rhiannonlynne closed 1 year ago

rhiannonlynne commented 1 year ago

A generalization of the UniformityMetric, beyond MJD. This was motivated because I saw someone calculating uniformity for a different derivable data column (I think it was phase for a periodic star), and they wrote their own version of the UniformityMetric.

rhiannonlynne commented 1 year ago

Ah, that's a good point. "1" == "good" and uniform, 0 == "bad" and nonuniform. Would it be too confusing to swap just this new one and not the old one? I am not sure if anyone else is using these results anywhere, but I would suspect not.

yoachim commented 1 year ago

DESC was looking at the Kuiper metric before and I think there was confusion since it was defined where smaller was better. That might be the only other one that needs flipping.

rhiannonlynne commented 1 year ago

I didn't flip the Kuiper metric (thought this might need more warning), but I did flip the "Uniformity" metrics here.