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

Downsample waveform in gated gaussian model #4940

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yi-fan-wang
Copy link
Member

@yi-fan-wang yi-fan-wang commented Nov 14, 2024

Standard information about the request

This is a: new feature

This change affects: inference

This change changes: scientific output

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

This change will: only affect the "gated_gaussian_noise" model in the inference.

Motivation

Siegel el at. showed in their recent paper the waveform should also be downsampled. So I add this feature for some tests. For nwo it's in gated gaussian model.

Contents

Add a new attribute waveform_srate in the static_params. The waveform would be generated in this sample rate and down sampled to the data's sample rate.

An example of the configuration file:

[data]
sample-rate = 1024

[static_params]
waveform_srate = 2048

With this configuration, the waveform would be generated with sampling rate 2048 Hz first and downsampled to 1024 Hz.

Links to any issues or associated PRs

Testing performed

Additional notes

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

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