rad_vel() Solar potential when observing the Sun in NOVAS C 3.1 #51
Comments
attipaci
added a commit
that referenced
this issue
Aug 1, 2024
attipaci
added a commit
that referenced
this issue
Aug 1, 2024
* v1.1.0 release composition (part 2) * Proper accounting of gravitational bending for radial velocities, and more. * Fix ccpcheck error in last commit * Another attempt to get deflected radial velocities right... * Tweaks and fill tesing holes * doxygen edits * Switch logic order in radial velocity calculation * Add rad_vel2() for radial velocity calculation with gravitational deflection. * Tweaks, clean up... * State typical (max.) radial velocity error from old calculation. * rad_vel() doxygen edits * Issues #50 and #51: rad_vel() handling of relativistic corrections * Issue #51: Tweak fix. * doxygen tweaks * Small CHANGELOG edit * Keep -devel in version for now...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the NOVAS C 3.1 implementation of
rad_vel(), the Solar gravitational potential (at the photosphere) was ignored when calculating radial velocities.Typically 'observing the Sun' means looking at its photosphere, which lies at a well defined location in the Sun' gravitational potential. As the light travels away from the Sun towards the observer, it is redshifted. The NOVAS C implementation of
rad_vel()has ignored this redshifting when the Sun was being observed. It affects SuperNOVAS releases prior to v1.1. also.The text was updated successfully, but these errors were encountered: