Skip to content

Commit

Permalink
Update scopesim/effects/ter_curves.py
Browse files Browse the repository at this point in the history
Co-authored-by: teutoburg <73600109+teutoburg@users.noreply.github.com>
  • Loading branch information
astronomyk and teutoburg committed Jan 30, 2024
1 parent 09a2adc commit 3690d25
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions scopesim/effects/ter_curves.py
Original file line number Diff line number Diff line change
Expand Up @@ -204,10 +204,12 @@ def plot(self, which="x", wavelength=None, *, axes=None, **kwargs):
wunit = params["wave_unit"]
# TODO: shouldn't need both, make sure they're equal
if wunit != wave_unit:
logger.warning(f"wavelength units in the meta dict of "
f"{self.meta.get('name')} are inconsistent: \n"
f"- wavelength_unit : {wave_unit} \n"
f"- wave_unit : {wunit}")
logger.warning("wavelength units in the meta dict of "

Check warning on line 207 in scopesim/effects/ter_curves.py

View check run for this annotation

Codecov / codecov/patch

scopesim/effects/ter_curves.py#L206-L207

Added lines #L206 - L207 were not covered by tests
"%s are inconsistent:\n"
"- wavelength_unit : %s\n"
"- wave_unit : %s",
{self.meta.get("name")},
wave_unit, wunit)

wave = np.arange(quantify(params["wave_min"], wunit).value,
quantify(params["wave_max"], wunit).value,
Expand Down

0 comments on commit 3690d25

Please sign in to comment.