Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@holtzmanjon
Copy link

we can look through these together if you want

many are just adding some extra parameters, e.g. to input the range over which to search in the xcorr

one significant change is the normalization

another is to just use the peak pixel in the xcorr for the RV, since this is just a starting guess for the fit. The fitting of the xcorr peak seems to be challenging across a broad range of spectra. The code is all still there.

also do a final xcorr as a useful diagnostic and to use to input to the gaussian decomposition routine for multiple peaks

turn off masking based on sky spectra only
change default to no gaussian smoothing for xcorr
raise specific exceptions rather than general
add raw ccpeak to xcorr output and use it for selecting initial model
add check for initial pars within bounds
fix major bug in multifit_lsq jacobian
   change step sizes for derivatives
add some memory cleanup
add xcorr of final model with each spectrum and put in output
modify to allow jointfit to run with single spectrum, for uniform interface
trap for failed initial RVs in jointfit, and remove these, but proceed with others
change weighting for initial paramters to be by S/N
change vscatter to use std and not mad
put in keywords for maxvel
add new continuum normalization to normalize obs and models the same way
change fine grid range
handling of single visit returns to make multi-visit
handling of failed frames in single visit
add some exception trapping
fix velocity array for CCF
add BC to CCF velocity array
calculate full CCF, but just look for peak in input range
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant