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

Production merge #200

Merged
merged 23 commits into from
Sep 17, 2024
Merged

Production merge #200

merged 23 commits into from
Sep 17, 2024

Conversation

shivankacker
Copy link
Member

@shivankacker shivankacker commented Aug 20, 2024

Dependent on ohcnetwork/ayushma#519


Changes

  • Adds support for GPT-4o models
  • Adds support for multi-model tests

Please test:

  • Chat functionality and quality of responses. (due to backend change for vector storage)
  • If tests are executing correctly with multiple models

ishanExtreme and others added 17 commits February 23, 2024 23:46
* error handling added

* error handling added
* Enhancements to pagination

* Update src/app/(main)/client.tsx

Co-authored-by: Shivank Kacker <[email protected]>

* Update src/app/admin/page.tsx

Co-authored-by: Shivank Kacker <[email protected]>

---------

Co-authored-by: Shivank Kacker <[email protected]>
* unnecessary packages removed

* package.json updated
* Upload questions from CSV implemented

* updated

* changes done

* in sync with main repo

* Bug fixes

* resolved

* pr Updated

* CSV Upload improved

* Ui update
* Stability Enhancements and Bug Fixes

* Build fixes

* Use react-markdown
* Cleaned up leftover chatbot code and switched to pnpm

* fix tanstack error

* configured corepack
Copy link

vercel bot commented Aug 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ayushma-fe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 13, 2024 10:00am

@gigincg
Copy link
Member

gigincg commented Aug 22, 2024

@aparnacoronasafe Can you test the updates to Ayushma, ready for Production Release

@samholics
Copy link

samholics commented Sep 3, 2024

  • Not able to register new users on staging. ( ERROR for site owner:
    Invalid domain for site key error is showing on Recaptcha field)
    WhatsApp Image 2024-09-03 at 17 38 46_d26fdaed

  • when stop recording without any audio inputs, its shows "You" or "Thanks for Watching" feedbacks
    WhatsApp Image 2024-09-03 at 17 37 41_1983eae7
    WhatsApp Image 2024-09-03 at 17 10 41_dd2d277a

@samholics
Copy link

Asked question : )My patient in Ards and suspected sepsis has MAP less than 65 and I gave him
Piperacillin tazobactam. What more antibiotics to be given if he still has fever on the 3rd day?

Answer - (2) cefepime (2 g q8h), or (3)
meropenem (1 g q8h) or imipenem-cilastatin (0.5 g q6h).
If the patient is allergic to ß-lactam antibiotics, use (1) aztreonam (2 g q8h) or (2) ciprofloxacin (400 mg
q12h) or
levofloxacin (750 mg q24h). Add vancomycin (loading dose of 25-30 mg/kg, then 15-20 mg/kg q8-12h) to
each of the above regimens.

  • Initial two times gave correct response as the answer mentioned above. but later Medicine name in the question transcripting to others and gave no response. (same question)
  • Audio output and the Green color highlighting of the text is not matching. text highlight is slower than audio.
  • for some scenarios transcription takes longer time than expected
  • Transcription Loading Icon should be more evident. users might think the page is frozen while its transcripting.
    https://drive.google.com/file/d/1zTf9_-iZVFcQ01iyEtQUy6xkvm6BmA6h/view?usp=drivesdk

@samholics
Copy link

Tested all the FAQs in PPH section and its all responding well.
note: Different Pronunciation of medical terms may result in NO results. (eg: PPH captured as BPH, piperacillin-tazobactam came as bifuraxilin lasopectum etc. )

Copy link

netlify bot commented Sep 7, 2024

Deploy Preview for ayushma-ohc ready!

Name Link
🔨 Latest commit cc73cd7
🔍 Latest deploy log https://app.netlify.com/sites/ayushma-ohc/deploys/66e40cd5ced7ed0008db9295
😎 Deploy Preview https://deploy-preview-200--ayushma-ohc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@shivankacker
Copy link
Member Author

@samholics

1 - Issue with recaptcha has been solved.
2 - Issue with empty audio is due to whisper hallucinating. Although model hallucinations are hard to prevent, I will cover it up in ohcnetwork/ayushma#521
3 - If the response was correct the first 2 times, an error on the third time would be due to hallucinations in the model. This should be a rare occurrence. If this happens often, we can cover it up later on.
4 - The audio highlights don't match currently. I will check and verify if it is possible to match them.
5 - Likely due to server load at OpenAI's end. I will investigate
6 - What do you suggest for making it more evident?

@gigincg I think we can merge to production as none of the pointed out issues pertain to the vector db change. They can be taken up later.

@gigincg gigincg merged commit 9278ec4 into production Sep 17, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

8 participants