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

Minor features and improvements #64

Merged
merged 8 commits into from
Jun 13, 2022
Merged

Minor features and improvements #64

merged 8 commits into from
Jun 13, 2022

Conversation

juanmc2005
Copy link
Owner

This PR addresses issue #57.

Changelog

  • Remove IrregularFileAudioReader
  • Remove file audio readers
  • Improve profiling implementation, there's now a profile() function that wraps an observable and a sequence of operators into an observable that tracks the processing time of each item
  • Remove possibility not to emit the corresponding waveform alongside the each prediction
  • Move feature pre-calculation from the pipeline to a new PrecalculatedFeaturesAudioSource that emits audio chunks and their corresponding segmentation and embeddings
  • In diart.stream, catch the event of the user closing the plot window and end execution properly
  • Make diart.blocks a package instead of a single file and group classes into categories for better organization
  • Clean some code

@juanmc2005 juanmc2005 added feature New feature or request API Improvements to the API refactoring Internal design improvements that don't change the API labels Jun 13, 2022
@juanmc2005 juanmc2005 added this to the Version 0.4 milestone Jun 13, 2022
@juanmc2005 juanmc2005 self-assigned this Jun 13, 2022
@juanmc2005 juanmc2005 merged commit b91550b into develop Jun 13, 2022
@juanmc2005 juanmc2005 deleted the feat/streams branch June 13, 2022 19:03
This was referenced Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Improvements to the API feature New feature or request refactoring Internal design improvements that don't change the API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant