search

lsst-ts / ts_phosim

High-Level Module to Perturb the PhoSim
GNU General Public License v3.0
0 stars 1 forks source link

[DM-39474] Update test data with new physical_filter name. #85

Closed jbkalmbach closed 1 year ago

suberlak commented 1 year ago

Looks good, thanks for the quick fix! As in Guillem's PR, it would be nice to paste the code snippet that was used to make the change, may help our future selves figure out what was done (and why) - do you have the link to the DM update that necessitated this PR? Thanks!

jbkalmbach commented 1 year ago

Yeah, here is the link to the commit inobs_lsst that caused this error: https://github.com/lsst/obs_lsst/commit/1ddb5105b04abe06364f2d4cd2bdc7c8530006f3

jbkalmbach commented 1 year ago

For the code I did the same thing as Guillem did just changing the filename to the one here in ts_phosim: https://github.com/lsst-ts/ts_wep/pull/186#issue-1732597379