-
Notifications
You must be signed in to change notification settings - Fork 18
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
Task #223069 & #223071 Fuzzball js integration done and added new fields in API for offline call #117
Task #223069 & #223071 Fuzzball js integration done and added new fields in API for offline call #117
Conversation
…lds in API for offline call
WalkthroughThe recent changes enhance the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant VoiceAnalyser
participant Fuzzball
participant LearnerAI
User->>VoiceAnalyser: Input audio
VoiceAnalyser->>VoiceAnalyser: Process audio
VoiceAnalyser-->>VoiceAnalyser: Retrieve non-denoised text
VoiceAnalyser-->>VoiceAnalyser: Retrieve denoised text
VoiceAnalyser->>Fuzzball: Compare text similarity
Fuzzball-->>VoiceAnalyser: Return similarity scores
VoiceAnalyser->>LearnerAI: Send selected text and metadata
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (2)
- package.json (1 hunks)
- src/utils/VoiceAnalyser.js (7 hunks)
Files skipped from review due to trivial changes (1)
- package.json
Additional comments not posted (5)
src/utils/VoiceAnalyser.js (5)
36-36
: Import statement forfuzzball
looks good.The import statement for the
fuzzball
library is correct and necessary for the similarity scoring functionality.
79-83
: State variable declarations look good.The state variables
nonDenoisedText
,denoisedText
, andisOfflineModel
are correctly initialized and necessary for managing the outputs and mode.
113-116
: Updates inhandleProcess
function look good.The
handleProcess
function correctly callsgetResponseText
and updates the state variablesnonDenoisedText
anddenoisedText
. Error handling is also appropriately managed.
489-496
: Logic for selectingresponse_text
looks good.The logic correctly uses
fuzz.ratio
to compare similarity scores and select the appropriate text, improving the accuracy of text retrieval.
503-506
: Updates to the payload structure look good.The payload structure in the
axios.post
call is correctly updated to includeresponse_text
,mode
, andpause_count
, allowing for richer contextual information to be transmitted.
Summary by CodeRabbit
New Features
Chores