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.

Sign up for GitHub

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

Jump to bottom

Add audio similarity operator #51

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Add audio similarity operator #51

wants to merge 2 commits into from

Conversation

duggalsu
Copy link
Collaborator

  • Added audio similarity operator
  • Synced feluda core requirements versions with audio operator packages
  • Synced text_vec_rep_paraphrase_lxml_requirements with new operator
  • Added SonarQube folder to gitignore

@duggalsu
Add audio similarity operator
744a69b 
- Added audio similarity operator
- Synced feluda core requirements versions with audio operator packages
- Synced text_vec_rep_paraphrase_lxml_requirements with new operator
- Added SonarQube folder to gitignore
@dennyabrain
Copy link
Contributor

@duggalsu can you make this operator adhere to the "interface" all operators have? they need to define an init and a run function. also creat a test_audio_similarity.py like the other operators where the operator is used so we can see how to use it. Another feedback is that can you add something in the name of the operator so its easy to grasp what technique or tool you are using to do audio similarity.

@duggalsu
Add audio operator to ES
c5a8f53 
- Partially added es code for audio similarity operator
- Renamed audio similarity features rmse operator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@duggalsu @dennyabrain