sensein / senselab

SenseLab is a Python package that simplifies building pipelines for biometric (e.g. speech, voice, video, etc) analysis.
https://github.com/sensein/senselab
Apache License 2.0
6 stars 3 forks source link

Task: Remove 'tasks' from import statements #39

Closed ibevers closed 1 month ago

ibevers commented 1 month ago

It doesn't seem informative, and it looks cleaner without it:

Example: from senselab.audio.preprocessing import resample_hf_dataset

is clearer than:

from senselab.audio.tasks.preprocessing import resample_hf_dataset

@fabiocat93 is there a reason to keep it in the import statements?

fabiocat93 commented 1 month ago

@ibevers, thanks for raising this point!

We may want to have tasks and workflows/pipelines (which are combinations of tasks). For example, the whisperx pipeline may combine automatic speech recognition, force alignment, and speaker diarization. Rn, we don't have any pipeline/workflow, but we will pretty soon

ibevers commented 1 month ago

Ah, right, I remember we talked about this now. Thanks!