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

ENH: Rewrite core.prep to use vocles + vocalpy, remove io module #558

Open
2 tasks
Tracked by #614
NickleDave opened this issue Jul 29, 2022 · 0 comments
Open
2 tasks
Tracked by #614

ENH: Rewrite core.prep to use vocles + vocalpy, remove io module #558

NickleDave opened this issue Jul 29, 2022 · 0 comments

Comments

@NickleDave
Copy link
Collaborator

NickleDave commented Jul 29, 2022

  • use vocalpy classes to convert audio -> spectrograms if specified
  • make a vocles dataset and save that by calling to_csv

if possible prefer clearly named functions, e.g. create_spectrograms_from_audio; current dataframe.from_files -> audio.to_spect -> spect.to_dataframe feels like concrete steps that can be written as statements but are obscured by calling out to multiple functions at different levels. It feels like there's 1 or 2 main pipelines we want to have built-in--spectrogram + audio datasets--and those should be written down and made explicit in code. Anything more exotic can be done by hand.

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

No branches or pull requests

1 participant