Skip to content
Snippets Groups Projects

Add missing GW docstrings

A bunch of functions/methods/classes in the GW sub-package are missing docstrings. This MR adds new docstrings to some of those and updates some others.

There are two deliberate code changes:

  • removing the call to the vectorized spin conversion when possible
  • removing the call to np.vectorize for redshift calculations that are no longer necessary after we pinned to astropy>=5

Closes #620 (closed)

Edited by Colm Talbot

Merge request reports

Pipeline #372317 passed

Pipeline passed for 86e4af2b on 620-docstring-for-bilby_to_lalsimulation_spins-should-specify-units

Test coverage 62.00% (0.00%) from 2 jobs
Approval is optional

Merged by Colm TalbotColm Talbot 3 years ago (Apr 4, 2022 7:23pm UTC)

Pipeline #379160 passed

Pipeline passed for bb8b244a on master

Test coverage 62.00% (0.00%) from 2 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading