Skip to content

Commit

Permalink
FWHM can use CCF errors consistently with RV
Browse files Browse the repository at this point in the history
  • Loading branch information
j-faria committed Jul 25, 2024
1 parent 7bea769 commit e667447
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions iCCF/iCCF.py
Original file line number Diff line number Diff line change
Expand Up @@ -295,10 +295,10 @@ def FWHM(self):
""" The full width at half maximum of the CCF """
eccf = self.eccf if self._use_errors else None
try:
return FWHMcalc(self.rv, self.ccf, self.eccf,
return FWHMcalc(self.rv, self.ccf, eccf,
guess_rv=self.pipeline_RV)
except ValueError:
return FWHMcalc(self.rv, self.ccf, self.eccf)
return FWHMcalc(self.rv, self.ccf, eccf)

@property
def FWHMerror(self):
Expand All @@ -324,7 +324,13 @@ def Wspan(self):
@cached_property
def contrast(self):
""" The contrast (depth) of the CCF, measured in percentage """
return contrast(self.rv, self.ccf)
return contrast(self.rv, self.ccf, self.eccf)

@cached_property
def contrast_error(self):
""" Uncertainty on the contrast (depth) of the CCF, measured in percentage """
return contrast(self.rv, self.ccf, self.eccf, error=True)


@property
def all(self):
Expand Down

0 comments on commit e667447

Please sign in to comment.