Skip to content

Adding Audification code - #5340

Open
Inorinnnnnn wants to merge 13 commits into
gwastro:masterfrom
Inorinnnnnn:master
Open

Adding Audification code#5340
Inorinnnnnn wants to merge 13 commits into
gwastro:masterfrom
Inorinnnnnn:master

Conversation

@Inorinnnnnn

Copy link
Copy Markdown

This is an addition of Audification of Gravitational Wave Script into PyCBC Library. It will not change any existing features

Standard information about the request

This is adding Audification Script into the PyCBC Library

This code will not affect any existing feature's.

This change: has appropriate unit tests, follows style guidelines (See e.g. PEP8), has been proposed using the contribution guidelines

This change will not break any function.

Motivation

This is not a changes but rather an addition to PyCBC Library for Audification of Gravitational Waves.

Contents

Adding pycbc_page_audio script into bin/plotting.

Links to any issues or associated PRs

#5177

  • The author of this pull request confirms they will adhere to the code of conduct

@GarethCabournDavies GarethCabournDavies left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that these comments are nothing to do with your project - they are simply changes to tidy up the code, match the style used in PyCBC, or avoid using code that already exists

A few places still need a bit of the conversion from plot_qscan

Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio
Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio Outdated
Comment thread bin/plotting/pycbc_page_audio Outdated
@GarethCabournDavies
GarethCabournDavies enabled auto-merge (squash) July 29, 2026 13:35
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.

2 participants