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

domain='auto' option for dsp.normalize? #572

Open
sikersten opened this issue Apr 9, 2024 · 0 comments
Open

domain='auto' option for dsp.normalize? #572

sikersten opened this issue Apr 9, 2024 · 0 comments
Labels
dsp enhancement For suggesting enhancements of current functionality good first issue

Comments

@sikersten
Copy link
Member

General

  • pyfar version: 0.6.5

Description

When using pyfar.dsp.normalize() with a FrequencyData object and default values, an error is thrown due to the default domain='time'. It would be nice to add an option domain='auto', which checks if signal is a FrequencyData object.

What I Did

import pyfar as pf
signal = pf.FrequencyData([1, 2, 3], [1, 2, 3])
pf.dsp.normalize(signal)
@sikersten sikersten added enhancement For suggesting enhancements of current functionality dsp good first issue labels Apr 9, 2024
@github-actions github-actions bot added this to To do in Code Backlog Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dsp enhancement For suggesting enhancements of current functionality good first issue
Projects
Development

No branches or pull requests

1 participant