Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redshift utilities #58

Merged
merged 15 commits into from
Aug 24, 2024
Merged

Redshift utilities #58

merged 15 commits into from
Aug 24, 2024

Conversation

attipaci
Copy link
Collaborator

@attipaci attipaci commented Aug 19, 2024

Add functions to handle redshift corrections to radial velocities.

  • grav_redshift() to calculate gravitational redshifts near massive bodies, for an observer far away.
  • redshift_vrad() and unredshift_vrad() to apply/undo redshift corrections to radial velocities.
  • novas_z_add() to compound redshift corrections.
  • novas_z_inv() to calculate a redshift for moving in the opposite direction.

@attipaci attipaci added the enhancement New feature or request label Aug 19, 2024
@attipaci attipaci added this to the 1.2.0 milestone Aug 19, 2024
@attipaci attipaci self-assigned this Aug 19, 2024
Copy link

codecov bot commented Aug 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.64%. Comparing base (369d569) to head (bbb7448).
Report is 2 commits behind head on v1.2.0.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           v1.2.0      #58   +/-   ##
=======================================
  Coverage   99.64%   99.64%           
=======================================
  Files           7        7           
  Lines        2805     2832   +27     
  Branches      542      546    +4     
=======================================
+ Hits         2795     2822   +27     
  Partials       10       10           
Flag Coverage Δ
unittests 99.64% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/novas.c 99.54% <100.00%> (-0.01%) ⬇️
src/super.c 100.00% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 369d569...bbb7448. Read the comment docs.

@attipaci attipaci force-pushed the redshift-functions branch from 5595e34 to fb8d056 Compare August 24, 2024 13:08
@attipaci attipaci merged commit f92a4e7 into v1.2.0 Aug 24, 2024
11 checks passed
@attipaci attipaci deleted the redshift-functions branch August 24, 2024 13:19
attipaci added a commit that referenced this pull request Aug 24, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Aug 29, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Sep 1, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Sep 4, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Sep 11, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Sep 15, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Sep 15, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Sep 19, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
attipaci added a commit that referenced this pull request Oct 28, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
@attipaci attipaci mentioned this pull request Oct 28, 2024
attipaci added a commit that referenced this pull request Nov 6, 2024
* Redshifted motion handling (#57)

* A corrected version ref in README

* Another reference to API documentation in README

* Small edits to README

* Shoft release schedule up to imporve Fedora package integration

* Add redshift handling routines.

* Add functions to apply/adjust gravitational redshift for radial velocities.

* Tweaks and redshift_add() function

* Tweaks

* Added novas_z_inv()

* Further tweaks

* small edits to error handling

* various fixes

* Small arithmetic simplification

* Doxygen edits

* Test cases for redshift functions

* Split out util.c

* test/Makefile: Fix bad rebase

* merge util.c -> super.c

* super.c: Delete empty lines
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant