not all `alg` functors have implementations for all four versions of `measure_rv_from_ccf`

RvSpectML / EchelleCCFs.jl

Computes Cross Correlation Functions (CCFs) with RvSpectML

https://rvspectml.github.io/RvSpectML-Overview/

MIT License

3 stars 4 forks source link

not all `alg` functors have implementations for all four versions of `measure_rv_from_ccf` #47

Open hematthi opened 2 years ago

hematthi commented 2 years ago

https://github.com/RvSpectML/EchelleCCFs.jl/blob/a639217e24ebc59282d42e27d739d3aea1a032c6/src/calc_rv/common.jl#L27-L57

MeasureRvFromCCFGaussian does not have version with covar_base argument MeasureRvFromCCFQuadratic does not have versions with ccf_covar argument and covar_base argument MeasureRvFromCCFCentroid only has version with ccf_var argument MeasureRvFromMinCCF only has first version

eford commented 2 years ago

Since trying to use something that doesn't exist will give an error, I removed bug label.